Skip to content

Relecture de contribution-guide.md#7

Merged
MachinisteWeb merged 3 commits intoworkingfrom
contribution-guide
Sep 16, 2017
Merged

Relecture de contribution-guide.md#7
MachinisteWeb merged 3 commits intoworkingfrom
contribution-guide

Conversation

@MachinisteWeb
Copy link
Member

Je m'attaque a celle là !

Signed-off-by: Bruno Lesieur <[email protected]>
Signed-off-by: Bruno Lesieur <[email protected]>
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

Une excellente façon de contribuer au projet est d'envoyer un rapport (« issue ») détaillé lorsque vous rencontrez un problème. Nous apprécions toujours un rapport de bogue bien écrit et nous vous en remercierons ! Avant de signaler un problème, lisez attentivement la documentation et vérifiez si votre problème n'a pas déjà été reporté ou résolu : [https://github.com/nuxt/nuxt.js/issues](https://github.com/nuxt/nuxt.js/issues)

## 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