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

Correction of quote for French + space before colon #3179

Merged
merged 2 commits into from
Jan 6, 2018

Conversation

Wadeck
Copy link
Contributor

@Wadeck Wadeck commented Dec 8, 2017

  • correction of a \' in french
  • add a space before :, in english only one space after, in french one before and one after

As the colon moved from HTML to translation (properties file), I let original translator validates the addition of colon in their language.

@reviewbybees

@Wadeck Wadeck added the needs-more-reviews Complex change, which would benefit from more eyes label Dec 8, 2017
@Wadeck Wadeck requested a review from alecharp December 8, 2017 09:27
@@ -26,7 +26,7 @@ CopyExisting=Copier un {0} existant
ItemName.help=Champ obligatoire
ItemName.label=Saisissez un nom
ItemName.validation.required=Ce champ ne peut pas \u00eatre vide. Veuillez saisir un nom valide et appuyer sur OK.
ItemType.validation.required=Veuillez choisir un type d\'Item
ItemType.validation.required=Veuillez choisir un type d\u2019Item
Copy link
Member

Choose a reason for hiding this comment

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

maybe we can translate "Item" as well

@Wadeck Wadeck requested a review from alecharp December 11, 2017 15:46
Copy link
Member

@alecharp alecharp left a comment

Choose a reason for hiding this comment

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

LGTM 🐝

@Wadeck Wadeck 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 Jan 2, 2018
@Wadeck
Copy link
Contributor Author

Wadeck commented Jan 2, 2018

@reviewbybees done

@oleg-nenashev
Copy link
Member

Hmm, it impacts other translations as well.
Let's wait for responses from CCed translation contributors

@oleg-nenashev oleg-nenashev added the on-hold This pull request depends on another event/release, and it cannot be merged right now label Jan 2, 2018
@damianszczepanik
Copy link
Member

As I see the colon has been moved from template to translations so it should not have impact for translations that were updated (hopefully all). So for Polish translation - fix is fine.

@Wadeck
Copy link
Contributor Author

Wadeck commented Jan 3, 2018

@damianszczepanik yeah all the translations where I can read the characters that have a colon were updated (and CC in the description)
Thank you for the review 👍

@oleg-nenashev oleg-nenashev removed the on-hold This pull request depends on another event/release, and it cannot be merged right now label Jan 6, 2018
@oleg-nenashev oleg-nenashev merged commit 93bf063 into jenkinsci:master Jan 6, 2018
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