-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Add Quarter period #36364
Add Quarter period #36364
Conversation
I was looking at the code, and noticed: julia/stdlib/Dates/src/query.jl Line 655 in e36b7b4
I think this is still wrong, for a leap year:
|
Good point! I think you could open an issue about it --- this is not something I have introduced with my changes. |
Thanks! |
Did you forget adding the following to adjusters.jl, or was it intentional?:
|
Indeed, I forgot to add these methods. Could you open a pull request to add them? |
done. #38277 |
This is the same as #35519, except that it is rebased (the only difference is to resolve the NEWS file conflict). Sorry to open a different pull request but I cannot modify the original pull request anymore — not sure what I did wrong.