-
-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
[JENKINS-72143] French translation #8578
[JENKINS-72143] French translation #8578
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Optional suggestions, good job!
Avertissement : ce plugin est compatible pour Jenkins {0} ou plus récent. \ | ||
Jenkins ne démarrera pas ce plugin s''il est installé. | ||
PluginManager.depCompatWarning=\ | ||
Avertissement : ce plugin nécessite des versions plus récentes de dépendances et au moins un de ces plugins n''est pas compatible avec la version installée. \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Avertissement : ce plugin nécessite des versions plus récentes de dépendances et au moins un de ces plugins n''est pas compatible avec la version installée. \ | |
Avertissement : ce plugin nécessite des versions plus récentes de dépendances et au moins un de ces dépendances n''est pas compatible avec la version installée. \ |
The link "dependency" = "plugin" isn't obvious, this equivalent formulation seems better to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/label localization |
Co-authored-by: Hervé Le Meur <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last suggestion fixing a typo, LGTM otherwise!
Co-authored-by: Hervé Le Meur <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/label ready-for-merge
This PR is now ready for merge. We will merge it after ~24 hours if there is no negative feedback.
Please see the merge process documentation for more information about the merge process.
Thanks!
/label hacktoberfest |
See JENKINS-72143.
Reported missing keys from hudson/Messages.properties to related Messages_fr.properties
Testing done
Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
Desired reviewers
@mention
Before the changes are marked as
ready-for-merge
:Maintainer checklist