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

Relecture de contribution-guide.md #7

Merged
merged 3 commits into from
Sep 16, 2017
Merged

Conversation

MachinisteWeb
Copy link
Member

Je m'attaque a celle là !

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>
Copy link

@forresst forresst left a 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

Choose a reason for hiding this comment

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

Je laisserai "Pull Requests"

Copy link
Member Author

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.

Copy link
Member Author

@MachinisteWeb MachinisteWeb Sep 16, 2017

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).

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 !

@MachinisteWeb MachinisteWeb merged commit 5b0b0a2 into working Sep 16, 2017
@MachinisteWeb MachinisteWeb deleted the contribution-guide branch September 16, 2017 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants