-
Notifications
You must be signed in to change notification settings - Fork 154
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
Editorial: Improve alignment between ISO 8601 vs. other-calendar operations #2506
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2506 +/- ##
=======================================
Coverage 95.49% 95.49%
=======================================
Files 20 20
Lines 10923 10923
Branches 2031 2031
=======================================
Hits 10431 10431
Misses 430 430
Partials 62 62 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Co-authored-by: Justin Grant <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the prose for CalendarFields needs one correction, the rest are just optional suggestions for improvements. Thanks for working on this, I think it's an improvement!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like everyone's review comments have been addressed. I'd like to merge this so we can keep other things moving along.
As for the notes that will need revision in #2485, I'll plan to rebase that on top of this when I merge it, and just remove the notes for now; if you have a suggestion for how they should be replaced instead, let me know over on that PR when you review it.
Extracted from #2500