Skip to content

Change kir /ky/ plural settings to same as for English#1634

Merged
ocean90 merged 5 commits into
GlotPress:developfrom
tobifjellner:tobifjellner-1633
Jun 9, 2023
Merged

Change kir /ky/ plural settings to same as for English#1634
ocean90 merged 5 commits into
GlotPress:developfrom
tobifjellner:tobifjellner-1633

Conversation

@tobifjellner

Copy link
Copy Markdown
Contributor

Fixes #1633

@tobifjellner

Copy link
Copy Markdown
Contributor Author

Are there any possible weird side-effects in various places when the plural parameters suddenly change?

@pedro-mendonca pedro-mendonca enabled auto-merge June 8, 2023 06:18
@pedro-mendonca pedro-mendonca disabled auto-merge June 8, 2023 06:18
@pedro-mendonca

Copy link
Copy Markdown
Member

For the number of plurals, is changing from 1 to 2, probably the strings with plurals that are "current" will keep that status, but will now be incomplete.

Translators and editors should be notified for this changelog, and if possible, the plural strings for the Locale should be marked as fuzzy.

@ocean90 ocean90 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you update the test to remove the now obsolete exception for kir in

Comment thread tests/phpunit/testcases/test_locales.php
`uz` needs to stay, only `kir` can be removed. Props: @ocean90
ocean90
ocean90 previously approved these changes Jun 8, 2023
Comment thread tests/phpunit/testcases/test_locales.php Outdated
Comment thread tests/phpunit/testcases/test_locales.php
@ocean90 ocean90 merged commit 907afbd into GlotPress:develop Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect plural settings for ky

3 participants