Skip to content
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

ICU-22720 Update ICU4J to be comparable for relative date formatting … #2950

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

garywade
Copy link

@garywade garywade commented Apr 4, 2024

…as ICU4C for days of the week and quarters

https://unicode-org.atlassian.net/browse/ICU-22720

Added enums for the days of week to RelativeUnit of RelativeDateTimeFormatter and changed QUARTERS to be included in this change rather than deprecated. Removed short-circuiting of unit tests for the comparable formatting in ICU4C. Added changes in formatNumericImpl and RelDateTimeDataSink to use the enums. Added unit tests to RelativeDateTimeFormatter.java to test the enums proposed and removed the short-circuiting of days-of-week tests.

Checklist
  • Required: Issue filed: https://unicode-org.atlassian.net/browse/ICU-22720
  • Required: The PR title must be prefixed with a JIRA Issue number.
  • Required: The PR description must include the link to the Jira Issue, for example by completing the URL in the first checklist item
  • Required: Each commit message must be prefixed with a JIRA Issue number.
  • Issue accepted (done by Technical Committee after discussion)
  • Tests included, if applicable
  • API docs and/or User Guide docs changed or added, if applicable

…as ICU4C for days of the week and quarters

Added enums for the days of week to RelativeUnit of RelativeDateTimeFormatter and changed QUARTERS to be included in this change rather than deprecated.  Removed short-circuiting of unit tests for the comparable formatting in ICU4C.  Added changes in formatNumericImpl and RelDateTimeDataSink to use the enums.  Added unit tests to RelativeDateTimeFormatter.java to test the enums proposed and removed the short-circuiting of days-of-week tests.
@garywade garywade self-assigned this Apr 4, 2024
@markusicu markusicu assigned richgillam and unassigned garywade Apr 11, 2024
Copy link
Contributor

@richgillam richgillam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants