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

Corrections tutoriel.rst #1610

Merged
merged 4 commits into from
Oct 20, 2014
Merged

Corrections tutoriel.rst #1610

merged 4 commits into from
Oct 20, 2014

Conversation

yapper-git
Copy link
Contributor

Corrections mineures de fautes de frappe et d'orthographe. Il en reste certainement...

Corrections mineures de fautes de frappe et d'orthographe. Il en reste certainement...
@@ -7,27 +7,27 @@ Les composantes d'un tutoriel

Un tutoriel n'est qu'un ensemble d'introduction de corps et de conclusion.
Copy link
Contributor

Choose a reason for hiding this comment

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

il manque une virgule ici non ?

@gustavi
Copy link
Contributor

gustavi commented Oct 20, 2014

J'ai relu de travers mais il reste pas mal quand même ! Peut être relire plus en détail avant de merger.

@yapper-git
Copy link
Contributor Author

J'ai corrigé d'autres typos. C'est ma première contribution sur GitHub (et à ZdS), j'espère que je m'y suis pris correctement ! Si vous pensez que mes corrections valent le coup, n'hésitez à les accepter. Merci pour ZdS ;-)

@Axylium
Copy link
Contributor

Axylium commented Oct 20, 2014

Normalement, pour les PR il y a une présentation à respecter, mais étant donné que ta PR ne touche que la doc et que c'est ta première, on va faire sans !

J'essayerai de lire tes corrections quand j'ai 5 minutes si quelqu'un d'autre n'est pas déjà passer, après il manquera plus que Travis (mais est-ce vraiment utile ?) et ça sera ok. :)

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling fda0b26 on yapper-git:patch-1 into 886d7ce on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) when pulling fda0b26 on yapper-git:patch-1 into 886d7ce on zestedesavoir:dev.

@SpaceFox
Copy link
Contributor

C'est OK pour moi, je merge !

Comme l'a dit @Florianboux, la prochaine fois pense à la présentation !

SpaceFox added a commit that referenced this pull request Oct 20, 2014
@SpaceFox SpaceFox merged commit 9c28fca into zestedesavoir:dev Oct 20, 2014
@SpaceFox SpaceFox added this to the Version 1.2 milestone Oct 20, 2014
@yapper-git yapper-git deleted the patch-1 branch October 20, 2014 17:54
@Eskimon
Copy link
Contributor

Eskimon commented Oct 21, 2014

@SpaceFox faudrait savoir, on met les milestones sur les PR ou pas finalement ?

@SpaceFox
Copy link
Contributor

Uniquement quand y'a pas de ticket (comme c'est le cas ici).

Je suis d'accord, c'est un peu confus, mais c'est le plus clair que j'ai trouvé au final...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants