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

Update English Translations #23233

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

Conversation

miq-bot
Copy link
Member

@miq-bot miq-bot commented Oct 15, 2024

Update the English Translations in the manageiq.po and manageiq.pot files.

@Fryguy
Copy link
Member

Fryguy commented Oct 15, 2024

@jrafanie Does it make sense to change the automated PRs to update the non-english message catalogs, particularly when a string is deleted? The reason that #23218 is failing is because the non-english catalogs still have the old strings, even though they've been deleted in the .pot and the english .po. I was hoping this PR would have those strings removed from the non-english catalogs, but that's not how it works currently.

@jrafanie
Copy link
Member

@jrafanie Does it make sense to change the automated PRs to update the non-english message catalogs, particularly when a string is deleted? The reason that #23218 is failing is because the non-english catalogs still have the old strings, even though they've been deleted in the .pot and the english .po. I was hoping this PR would have those strings removed from the non-english catalogs, but that's not how it works currently.

Maybe but that's not how the workflow occurs. We don't ever really merge the non-english po files except from translators. If we remove things in the english pot, it should be removed when the translators apply the changes and provide the updated po files.

@miq-bot
Copy link
Member Author

miq-bot commented Nov 13, 2024

Checked commit miq-bot@bd948a0 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 🍪

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.

3 participants