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

[bot] Remove obsolete l10n strings and ftl files #15496

Closed
2 of 3 tasks
github-actions bot opened this issue Nov 14, 2024 · 0 comments · Fixed by #15770
Closed
2 of 3 tasks

[bot] Remove obsolete l10n strings and ftl files #15496

github-actions bot opened this issue Nov 14, 2024 · 0 comments · Fixed by #15770
Assignees
Labels
Frontend HTML, CSS, JS... client side stuff L10N

Comments

@github-actions
Copy link

github-actions bot commented Nov 14, 2024

Remove obsolete strings

Search the code base for strings marked obsolete that are more than 2 months old.
If you find strings that have no date, either find the date by looking at git history
or add an expiry date and leave it to be removed in 2 months.

  • Remove old obsolete strings and their fallback usage

Remove obsolete ftl files

Compare the files in /l10n/en-US/ to the /en/ folder in the www-l10n repo and remove
files from www-l10n that no longer exist in bedrock.

If you have both repos installed and up to date you can run this command:
diff -r www-l10n/en bedrock/l10n/en | grep ".ftl"

  • Remove files
  • Remove references in www-l10n/configs/pontoon.toml
@github-actions github-actions bot added Frontend HTML, CSS, JS... client side stuff L10N labels Nov 14, 2024
@stephaniehobson stephaniehobson self-assigned this Dec 18, 2024
stephaniehobson added a commit that referenced this issue Dec 19, 2024
* Remove obsolete l10n strings (Fix #15496)

* Hard code CTD page title translations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend HTML, CSS, JS... client side stuff L10N
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant