Skip to content

Conversation

@ChrisDryden
Copy link
Collaborator

This is the approach I would use by just directly using jiff-icu to perform the conversions and not having to create new icu dates to get the ICU localizations. I think this approach is much more extensible to be able to cover all of the different formatters and modifiers.

I also added a few more regression tests for the localizations that we have so far so that we don't regress with the localizations we've made already as we add more features

@sylvestre
Copy link
Contributor

i am now ashamed of my previous iteration :)
well done!

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/shuf/shuf-reservoir (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/sort/sort-stale-thread-mem (passes in this run but fails in the 'main' branch)

@ChrisDryden ChrisDryden marked this pull request as ready for review January 26, 2026 20:51
@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/tail/retry. tests/tail/retry is passing on 'main'. Maybe you have to rebase?
Skipping an intermittent issue tests/shuf/shuf-reservoir (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/sort/sort-stale-thread-mem (passes in this run but fails in the 'main' branch)

@ChrisDryden ChrisDryden force-pushed the jiff-icu-locale-calendar branch from d3d356c to fdf3ea9 Compare January 28, 2026 17:15
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/shuf/shuf-reservoir (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/sort/sort-stale-thread-mem (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tail/inotify-dir-recreate (passes in this run but fails in the 'main' branch)

@sylvestre sylvestre merged commit efbe49f into uutils:main Jan 28, 2026
170 of 171 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants