Skip to content

translation of filecmp.po #618

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

Jump to bottom
Merged
merged 18 commits into from
Mar 29, 2019
Merged

translation of filecmp.po #618

merged 18 commits into from
Mar 29, 2019

Conversation

@Zepmanbc
Copy link
Contributor

@Zepmanbc Zepmanbc commented Mar 21, 2019

No description provided.

Seluj78
Seluj78 requested changes Mar 21, 2019
View reviewed changes
Copy link
Collaborator

@Seluj78 Seluj78 left a comment

Choose a reason for hiding this comment

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

Merci beaucoup pour ta contribution ! Je ne l'ai relue que pour les erreurs RST. Il faut que tu passe powrap sur le fichier dit travis ;)
@Zepmanbc
transtale filecmp
b1a70e9 
ajout des espaces avant les ':' passage par `powrap`
@Zepmanbc
Copy link
Contributor Author

Zepmanbc commented Mar 22, 2019

🎉 le build est passé
@Zepmanbc
Copy link
Contributor Author

Zepmanbc commented Mar 22, 2019

comment ça se passe après? je dois faire quelque chose d'autre? c'est toi qui ferme mon issue?
@Seluj78
Copy link
Collaborator

Seluj78 commented Mar 22, 2019

Alors du coup on va relire encore, perso j'ai relu principalement pour la syntaxe. Une fois que tout le monde est daccord et approuvent la PR alors @JulienPalard ou @christopheNan merge
christopheNan
christopheNan suggested changes Mar 22, 2019
View reviewed changes
Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

Merci pour cette très bonne contribution. Juste quelques remarques.
christopheNan and others added 9 commits March 23, 2019 09:15
@christopheNan @Zepmanbc
Update library/filecmp.po
1d967ed 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@christopheNan @Zepmanbc
Update library/filecmp.po
150d352 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@christopheNan @Zepmanbc
Update library/filecmp.po
6153afd 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@christopheNan @Zepmanbc
Update library/filecmp.po
f849e6d 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@christopheNan @Zepmanbc
Update library/filecmp.po
f82dfda 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@christopheNan @Zepmanbc
Update library/filecmp.po
624d92d 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@christopheNan @Zepmanbc
Update library/filecmp.po
3582bc5 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@christopheNan @Zepmanbc
Update library/filecmp.po
81b2232 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@Zepmanbc
translate filecmp.po
b215a3f 
modification suite au retour de christopheNan
JulienPalard
JulienPalard requested changes Mar 26, 2019
View reviewed changes
Copy link
Member

@JulienPalard JulienPalard left a comment

Choose a reason for hiding this comment

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

LGTM with nits
@JulienPalard @Zepmanbc
Update library/filecmp.po
b99d0bf 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@Seluj78
Copy link
Collaborator

Seluj78 commented Mar 27, 2019

@Zepmanbc gaffe, github a plein de suggestions marquées en outdated. Vérifie qu'elles ai été implémentées avant de les mettre en resolved
@Zepmanbc
Copy link
Contributor Author

Zepmanbc commented Mar 27, 2019

@Seluj78 j'ai appliqué toutes les modifications à la main, je n'ai pas cliqué sur Commit suggestion à chaque fois. Il fallait?
Quelle est la bonne façon de faire? (il existe un guide du genre "les bonnes pratiques du commiteur" ?)
@Seluj78
Copy link
Collaborator

Seluj78 commented Mar 27, 2019

@Zepmanbc https://help.github.com/en/articles/incorporating-feedback-in-your-pull-request#applying-a-suggested-change ;)
@christopheNan @Zepmanbc
Update library/filecmp.po
399ee24 
Co-Authored-By: Zepmanbc <zepman@gmail.com>
@Zepmanbc
Copy link
Contributor Author

Zepmanbc commented Mar 27, 2019

@Seluj78 merci
Donc soit j'utilise le bouton [Commit suggestion], soit je le fait "à la main" et je clique sur [Resolve conversation]
normalement j'ai fait le tour
la prochaine ira mieux, je serai un peu moins boulet normalement ^^
@Seluj78
Copy link
Collaborator

Seluj78 commented Mar 27, 2019

yes exactement ! Et si tu va dans l'onglet files changed de github tu peux appliquer les suggestions en batch :D
christopheNan
christopheNan approved these changes Mar 27, 2019
View reviewed changes
Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

Très bonne contribution. Merci !
@JulienPalard
Copy link
Member

JulienPalard commented Mar 29, 2019

Top ! Merci pour cette PR \o/
@JulienPalard JulienPalard merged commit 5238d09 into python:3.7 Mar 29, 2019
@Seluj78 Seluj78 mentioned this pull request Apr 11, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants

@Zepmanbc @Seluj78 @JulienPalard @christopheNan