Skip to content

Conversation

@Zepmanbc
Copy link
Contributor

@Zepmanbc Zepmanbc commented May 9, 2019

closes #715

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 contribution.
Quelques remarques.

Zepmanbc and others added 4 commits May 11, 2019 19:25
 correction suite à la review de ChristopheNan Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@Seluj78 Seluj78 requested a review from christopheNan May 12, 2019 18:51
@Seluj78
Copy link
Collaborator

Seluj78 commented May 12, 2019

@Zepmanbc j'ai unresolve plein de suggestions de @christopheNan que tu n'avais pas appliquées :)

@Zepmanbc
Copy link
Contributor Author

@Seluj78 je n'ai pas encore trouve la "bonne" façon d'appliquer les modifs et de valider les suggestions, tu as une méthodes à conseiller? un tuto?

@Seluj78
Copy link
Collaborator

Seluj78 commented May 13, 2019

Il y a un bouton ;)

Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@Seluj78
Copy link
Collaborator

Seluj78 commented May 13, 2019

yay 🎉 😉

@Zepmanbc
Copy link
Contributor Author

il y a pleins de boutons ^^ , mais là je fais un commit et apres je dois récupérer pour faire powrap, c'est pas terrible de devoir le faire en 2 temps, Travis bosse dans le vent

@Seluj78
Copy link
Collaborator

Seluj78 commented May 13, 2019

ouais c'est vrai mais sinon tu appliqe toutes les suggestions en batch et tu fait powrap a la fin, pas besoin de le faire a chaques fois
Et tkt pour travis :)

@JulienPalard JulienPalard merged commit e895e6b into python:3.7 May 30, 2019
@JulienPalard
Copy link
Member

@Zepmanbc Merci pour cette trad' :)

@Zepmanbc Zepmanbc deleted the numbers branch May 1, 2020 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants