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

I18n script runner #2781

Closed
wants to merge 2 commits into from
Closed

Conversation

nichwall
Copy link
Contributor

This PR includes 2 new workflows:

  • i18n-auto-branch which automatically runs the key-copying script on pushes to the main/master branch and opens a branch with the updated files if there were any changes
  • i18n-manual-branch which can be manually triggered

The branch workflows are identical other than the trigger condition to allow for disabling the automatic workflow without removing the ability to run the manual workflow.

These two workflows will need write access to be able to create the branch and PR. These also require defining the USER_EMAIL and USER_NAME secrets for the git initialization. These values don't necessarily need to be secrets, but felt better to do that instead of just including a default value in the action.

These workflows make use of the https://github.com/audiobookshelf/audiobookshelf-i18n-updater GitHub action. The script this action runs ensures a newline at the end of each i18n file so the last line stops showing up as a change when viewing the diff when the number of lines changes in an i18n file.

@nichwall
Copy link
Contributor Author

This workflow is no longer needed due to Weblate being used to manage translations.

@nichwall nichwall closed this Jun 21, 2024
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.

1 participant