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

Delete autogenerated locale files and gitignore them #288

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

riknoll
Copy link
Member

@riknoll riknoll commented Apr 30, 2024

The string files for each of the libs are generated automatically by the build, so there is no actual reason to check them in. This adds them to the gitignore and deletes them from the repo. The motivation for this is that crowdin upload is failing because a few of the checked in filenames have changed case (e.g. V1-strings.json -> v1-strings.json) and crowdin was throwing an error when we tried to upload the new versions with the checked in versions. I guess crowdin doesn't allow files with the same name but different casing.

This PR doesn't touch the hand translated strings in libs/core/_locales/de/*. Those should be checked in.

@riknoll riknoll requested review from Amerlander, jwunderl and a team April 30, 2024 22:04
@riknoll riknoll merged commit aca22ba into master Apr 30, 2024
6 of 7 checks passed
@riknoll riknoll deleted the dev/riknoll/remove-locales2 branch April 30, 2024 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants