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

🚚 Stop normalizing YAML and PO files #5337

Merged
merged 2 commits into from
Mar 29, 2024
Merged

🚚 Stop normalizing YAML and PO files #5337

merged 2 commits into from
Mar 29, 2024

Conversation

rix0rrr
Copy link
Collaborator

@rix0rrr rix0rrr commented Mar 29, 2024

Our plans to finagle Weblate into committing only once per day to reduce conflicts, allowing us to heavily normalize the translation files, has failed. Weblate is going to commit constantly, and it will have created files with the headers, wrapping, and key ordering that it has decided on its end. Any of our attempts to reformat those files will just lead to merge conflicts.

Instead: turn off YAML and POfile formatting; instead what we do to minimize the chances of conflicts is to stop PyBabel from updating PO file headers. Only Weblate is allowed to update those headers.

rix0rrr and others added 2 commits March 29, 2024 14:52
Our plans to finagle Weblate into committing only once per day to reduce
conflicts, allowing us to heavily normalize the translation files, has
failed. Weblate is going to commit constantly, and it will have created
files with the headers, wrapping, and key ordering that it has decided
on its end. Any of our attempts to reformat those files will just lead
to merge conflicts.

Instead: turn off YAML and POfile formatting; instead what we do to
minimize the chances of conflicts is to stop PyBabel from updating
PO file headers. Only Weblate is allowed to update those headers.
@rix0rrr rix0rrr merged commit cd8d7b8 into main Mar 29, 2024
10 checks passed
@rix0rrr rix0rrr deleted the stop-normalizing branch March 29, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant