-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set first day of week #31
base: master
Are you sure you want to change the base?
Conversation
This pull request also includes the pull request for Support more than 5 colors. Also it adds the package-lock.json file, which isn't included in the repository currently. (I don't know if this is intended or not, I'm neither the maintainer, nor do I use this package actively myself.) But appart from these, I'd love to see this merged. |
@SteffoSpieler thanks for the review. I would create a different branch and cherry pic the related changes if the Maintainer is willing to merge it in the first place. ( cherry picking resulted in merge conflicts since I changed the constructor in both PRs I think) |
as far as I found out |
Since in most parts of the world the week does not start with sunday I implemented this.