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

Use new pre-commit script to detect and fix existing i18n syntax issues #9486

Closed
6 tasks done
rebecca-shoptaw opened this issue Jun 25, 2024 · 1 comment
Closed
6 tasks done
Assignees
Labels
Lead: @mekarpeles Issues overseen by Mek (Staff: Program Lead) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Theme: Internationalization Making OpenLibrary work for both foreign-language users and books. [managed] Type: Epic A feature or refactor that is big enough to require subissues. [managed] Type: Feature Request Issue describes a feature or enhancement we'd like to implement. [managed]

Comments

@rebecca-shoptaw
Copy link
Collaborator

rebecca-shoptaw commented Jun 25, 2024

Problem

The new detect_missing_i18n script by @pidgezero-one has given us a super helpful list of i18n syntaxing issues throughout the codebase and/or valid i18n that accidentally gets flagged by the script.

These files are listed in EXCLUDE_LIST in detect_missing_i18n.py.

Proposal & Constraints

Once the PR (#9428) is merged, I propose going through the full list of excluded files myself and making any changes where needed, either fixing i18n syntax or adjusting distribution of text to avoid accidental flags of the script. I'll also make any updates to the i18n Developer's Guide where needed as a result of edge-cases I discover in the process.

Related files

detect_missing_i18n.py

Issue phases & sub-PRs

Stakeholders

@pidgezero-one


Instructions for Contributors

  • Please run these commands to ensure your repository is up to date before creating a new branch to work on this issue and each time after pushing code to Github, because the pre-commit bot may add commits to your PRs upstream.
@rebecca-shoptaw rebecca-shoptaw added Type: Feature Request Issue describes a feature or enhancement we'd like to implement. [managed] Theme: Internationalization Making OpenLibrary work for both foreign-language users and books. [managed] Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] Needs: Lead labels Jun 25, 2024
@rebecca-shoptaw rebecca-shoptaw self-assigned this Jun 25, 2024
@rebecca-shoptaw rebecca-shoptaw added the Type: Epic A feature or refactor that is big enough to require subissues. [managed] label Jun 28, 2024
@mekarpeles mekarpeles added Priority: 3 Issues that we can consider at our leisure. [managed] Lead: @mekarpeles Issues overseen by Mek (Staff: Program Lead) [managed] and removed Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] Needs: Lead labels Jul 1, 2024
@rebecca-shoptaw
Copy link
Collaborator Author

All subtasks complete and merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Lead: @mekarpeles Issues overseen by Mek (Staff: Program Lead) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Theme: Internationalization Making OpenLibrary work for both foreign-language users and books. [managed] Type: Epic A feature or refactor that is big enough to require subissues. [managed] Type: Feature Request Issue describes a feature or enhancement we'd like to implement. [managed]
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants