Skip to content

Conversation

@allejok96
Copy link
Contributor

A lot of strings from Carla have been uploaded to our Transifex. There's no use translating these since Carla is a separate process and doesn't read our translation files. This PR adds git ls-files to only include strings from our files.

@allejok96
Copy link
Contributor Author

Copy link
Member

@messmerd messmerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@allejok96
Copy link
Contributor Author

So I just discovered a second repo where the latest Transifex translations are auto committed by a bot. @liushuyu then takes those translations and merges them into LMMS every now and then.

This bug is also present in that repo's scripts. It feels bad to spread our tools in two places. IMO as much as possible should be in this repo, and only the stuff related to bot committing should be in that second repo.

https://github.com/liushuyu/lmms-translations

@allejok96
Copy link
Contributor Author

So just as a reminder, even if this fix gets merged

  1. @liushuyu will need to fix this bug in whatever scripts they may be using in the process.
  2. @liushuyu will need to regenerate en.ts and upload it to Transifex to get rid of all the Carla strings there.

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