Skip to content

add 'make dump-translations' #29825

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 12 commits into from
Closed

add 'make dump-translations' #29825

wants to merge 12 commits into from

Conversation

silverwind
Copy link
Member

@silverwind silverwind commented Mar 15, 2024 •
edited
Loading

This script will help us working with the translations. Right now it only has one command dump. Later on, we can use it to lint the translations in the templates/go code. Example run:

$ make dump-translations | tail -10
actions.variables.update.success
projects.type-1.display_name
projects.type-2.display_name
projects.type-3.display_name
filemode.changed_filemode
filemode.directory
filemode.normal_file
filemode.executable_file
filemode.symbolic_link
filemode.submodule

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 15, 2024
@silverwind silverwind added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Mar 15, 2024
@silverwind
Copy link
Member Author

I think I may extend this now with linting-style commands so that it can find unused or missing translations.

@silverwind silverwind marked this pull request as draft March 15, 2024 20:56
@silverwind silverwind mentioned this pull request Mar 15, 2024
Closed
@silverwind silverwind mentioned this pull request Mar 17, 2024
Merged
@silverwind
Copy link
Member Author

I think I will move this into a future translations linter.

@silverwind silverwind closed this Apr 13, 2024
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Jul 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/dependencies modifies/internal modifies/js skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@silverwind @GiteaBot