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

translation-tool.pl - Feature for adding unique prefix for each missing translation #2632

Merged
merged 1 commit into from
Nov 27, 2016
Merged

translation-tool.pl - Feature for adding unique prefix for each missing translation #2632

merged 1 commit into from
Nov 27, 2016

Conversation

damianszczepanik
Copy link
Member

Motivation and advantages are presented on attached screenshot. Sometimes I found hard to find where is the proper *.properties file with translation so I added feature for provided translation-tool.sh script that marks each missing translation by:

  • adding TranslateMe prefix so it is easily found on the page
  • unique number so developer can in a second find where is the string that should be translated

@damianszczepanik
Copy link
Member Author

translate

@oleg-nenashev oleg-nenashev added the needs-more-reviews Complex change, which would benefit from more eyes label Nov 13, 2016
@oleg-nenashev
Copy link
Member

LGTM 👍

@damianszczepanik
Copy link
Member Author

@daniel-beck can you merge this change or you have some comments/questions ?

@daniel-beck
Copy link
Member

I'm not really qualified to evaluate this change, knowing little Perl.

Besides that, LGTM, so assuming nobody else objects I suppose we can merge in a few days. It's not like it'll break Jenkins even if it doesn't work at all ;-)

@batmat
Copy link
Member

batmat commented Nov 26, 2016

👍 I suppose .pl stands for polish ;-)

@oleg-nenashev oleg-nenashev added ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback and removed needs-more-reviews Complex change, which would benefit from more eyes labels Nov 27, 2016
@oleg-nenashev oleg-nenashev merged commit f4d3ec2 into jenkinsci:master Nov 27, 2016
@oleg-nenashev
Copy link
Member

I'm not sure if it requires changelogs. Maybe this tool needs to be decoupled to a separate repo

oleg-nenashev added a commit that referenced this pull request Nov 27, 2016
@damianszczepanik damianszczepanik deleted the translation-tool branch November 27, 2016 10:56
@damianszczepanik
Copy link
Member Author

I'm using this tool when adding new translations. I found eg that the order of translated keys is not the same as in original. I will try to look deeper at this and maybe prepare new patchset.

I don't think that moving to separate repo is good idea because then new developers may not be aware that there is dedicated tool that helps them with translating.

@oleg-nenashev
Copy link
Member

@damianszczepanik Maybe a blogpost or jenkins.io article could be also helpful to highlight such tools.Maybe a subproject like https://jenkins.io/projects/infrastructure/

@daniel-beck
Copy link
Member

daniel-beck added a commit that referenced this pull request Nov 27, 2016
It's not actually a change in anything we deliver.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback
Projects
None yet
4 participants