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

Correction (orthographe) #3756

Merged
merged 1 commit into from
Jul 31, 2016
Merged

Correction (orthographe) #3756

merged 1 commit into from
Jul 31, 2016

Conversation

helmasaur
Copy link
Contributor

@helmasaur helmasaur commented Jul 31, 2016

@vhf
Copy link
Contributor

vhf commented Jul 31, 2016

Super, merci beaucoup ! :)

@vhf vhf merged commit 711094a into zestedesavoir:dev Jul 31, 2016
@GerardPaligot
Copy link
Member

@helmasaur Merci pour cette première contribution !

Pour la prochaine fois, remplit le template de la PR par rapport à ta proposition de correctif. ^^

@vhf vhf added this to the Version de développement milestone Jul 31, 2016
@helmasaur helmasaur deleted the patch-1 branch July 31, 2016 20:05
@helmasaur
Copy link
Contributor Author

La PR, c'est le tableau ? J'étais pas sûr donc j'ai préféré supprimer plutôt que de faire une erreur.

@coveralls
Copy link

coveralls commented Jul 31, 2016

Coverage Status

Coverage increased (+34.8%) to 87.685% when pulling ada85ea on Helmasaur:patch-1 into df94224 on zestedesavoir:dev.

@GerardPaligot
Copy link
Member

@helmasaur Pas d'inquiétude. Il suffit de remplacer les valeurs de droite du tableau et la procédure de QA sur la liste à puce.

Pour info, une procédure de QA est la procédure que doit appliquer le contributeur sur son instance locale de Zeste de Savoir pour tester ta PR et la valider. En l'occurrence, pour cette PR précise, la procédure de QA pouvait se limiter à faire de la revue de code. :)

@helmasaur
Copy link
Contributor Author

D'accord, merci pour ces informations !

@vhf vhf mentioned this pull request Aug 23, 2016
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.

4 participants