Skip to content

Relecture de contribution-guide.md #7

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.

Sign up for GitHub

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

Merged
merged 3 commits into from
Sep 16, 2017
Merged

Relecture de contribution-guide.md #7

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
00eb8b9
Relecture du guide de contribution
MachinisteWeb Sep 15, 2017
911210d
Bon pour moi
MachinisteWeb Sep 15, 2017
86751f1
Aligner les titres
MachinisteWeb Sep 15, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions en/guide/contribution-guide.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,19 +1,19 @@
---
title: Contribution Guide
description: Any contribution to Nuxt.js is more than welcome!
title: Guide de contribution
description: Toute contribution à Nuxt.js est plus que bienvenue !
---

> Any contribution to Nuxt.js is more than welcome!
> Toute contribution à Nuxt.js est plus que bienvenue !

## Reporting Issues
## Rapports de problème

A great way to contribute to the project is to send a detailed report when you encounter an issue. We always appreciate a well-written bug report, and will thank you for it! Before reporting an issue, please read carefully the documentation and search if any issue for your problem doesn't already exist: https://github.com/nuxt/nuxt.js/issues
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
Copy link

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 •
edited
Loading

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

Copy link

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 !


We'd love to see your pull requests, even if it's just to fix a typo. Any significant improvement should be documented as [a GitHub issue](https://github.com/nuxt/nuxt.js/issues) before anybody starts working on it.
Nous adorerions voir vos propositions de fusion (« pull requests »), même s'il s'agit uniquement de corrections orthographiques. Toute amélioration significative devrait être documentée dans [un rapport détaillé sur GitHub](https://github.com/nuxt/nuxt.js/issues) avant que quelqu'un ne commence à y travailler.

### Convention

- For a fix, the branch name should be `fix-XXX` where XXX is the issue number or the name of what your fix does
- For a feature, the branch name should be `feature-XXX` where XXX is the issue number associated to this feature request
- Pour un correctif, le nom de la branche devrait être `fix-XXX` où `XXX` est le numéro du rapport correspondant ou le nom de votre correctif.
- Pour une fonctionnalité, le nom de la branche doit être `feature-XXX` où `XXX` est numéro du rapport correspondante à cette demande de fonctionnalité.