-
Notifications
You must be signed in to change notification settings - Fork 11
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
Relecture de contribution-guide.md
#7
Conversation
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
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.
Une remarque à rediscuter si besoin
|
||
## Pull Requests | ||
## Propositions de fusion |
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.
Je laisserai "Pull Requests"
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.
Je vérifie juste que j'ai bien précisé dans le corp du texte.
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.
Oui, comme pour SPA, c'est précisé à la ligne juste dessous.
En l'état « Pull Request » veut dire « Je fais une « requête (request) » pour que vous acceptiez de « rappatrier (pull) » mon code dans le votre.
Dans Git le fait de merger deux branche s'appelle une « fusion » en français.
Source : https://git-scm.com/book/fr/v1/Les-branches-avec-Git-Brancher-et-fusionner%C2%A0%3A-les-bases
L'onglet « Pull Requests » propre à GitHub permet, avant fusion, de proposer la fusion afin de permettre aux autres d'accepter ou de refuser la fusion. Cela me semble une traduction appropriée. Je l'ai pour ma part adoptée (même s'il est vrai qu'il n y a pas de documentation française GitHub officielle) :
— https://blog.lesieur.name/comprendre-et-utiliser-git-avec-vos-projets/
Si tu as une traduction plus élégante je veux bien prendre, mais je ne souhaite pas conserver un titre anglais pour quelque chose que l'on peut traduire facilement.
Qu'en pense tu ? Qu'en pense les autres ?
Pour ma part je souhaiterais conserver ce terme (en précisant « pull request » à la première occurance dans le texte qui l'utilise, pas dans le titre).
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.
Tu m'as convaincu, ça me va !
Je m'attaque a celle là !