-
Notifications
You must be signed in to change notification settings - Fork 161
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
Corrections tutoriel.rst #1610
Conversation
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. |
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.
il manque une virgule ici non ?
J'ai relu de travers mais il reste pas mal quand même ! Peut être relire plus en détail avant de merger. |
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 ;-) |
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. :) |
C'est OK pour moi, je merge ! Comme l'a dit @Florianboux, la prochaine fois pense à la présentation ! |
@SpaceFox faudrait savoir, on met les milestones sur les PR ou pas finalement ? |
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... |
Corrections mineures de fautes de frappe et d'orthographe. Il en reste certainement...