Skip to content

Remove and ignore English nvda.po as it is not needed, yet is provided by Crowdin.#15989

Merged
seanbudd merged 1 commit into
betafrom
noEnglish
Jan 1, 2024
Merged

Remove and ignore English nvda.po as it is not needed, yet is provided by Crowdin.#15989
seanbudd merged 1 commit into
betafrom
noEnglish

Conversation

@michaelDCurran

Copy link
Copy Markdown
Member

Link to issue number:

Mentioned in pr #15968

Summary of the issue:

When exporting translated po files from Crowdin, English is also included. this is not needed and just takes up space in the repository.

Description of user facing changes

None

Description of development approach

  • Remove the accidentally committed English nvda.po
  • Add English nvda.po to .gitignore so that it is not committed again.

Testing strategy:

File is not known to NVDA.

Known issues with pull request:

None known.

Code Review Checklist:

  • Documentation:
    • Change log entry
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • API is compatible with existing add-ons.
  • Security precautions taken.

@michaelDCurran michaelDCurran requested a review from a team as a code owner December 30, 2023 21:35
@michaelDCurran michaelDCurran requested review from gerald-hartig and removed request for a team December 30, 2023 21:35
@seanbudd seanbudd added the conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. label Jan 1, 2024
@seanbudd seanbudd merged commit 97aa6ca into beta Jan 1, 2024
@seanbudd seanbudd deleted the noEnglish branch January 1, 2024 23:40
@nvaccessAuto nvaccessAuto added this to the 2024.2 milestone Jan 1, 2024
@seanbudd seanbudd modified the milestones: 2024.2, 2024.1 Jan 10, 2024
Adriani90 pushed a commit to Adriani90/nvda that referenced this pull request Mar 13, 2024
…d by Crowdin. (nvaccess#15989)

Mentioned in pr nvaccess#15968

Summary of the issue:
When exporting translated po files from Crowdin, English is also included. this is not needed and just takes up space in the repository.

Description of user facing changes
None

Description of development approach
Remove the accidentally committed English nvda.po
Add English nvda.po to .gitignore so that it is not committed again.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants