Skip to content

Conversation

@Seluj78
Copy link
Collaborator

@Seluj78 Seluj78 commented Nov 14, 2018

No description provided.

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.

Énormément de conversations de @awecx étaient marquées commes résolues mais ne l'étaient pas, fait attention à ça, ce n'est pas la première fois que ça t'arrive.

JulienPalard and others added 2 commits November 29, 2018 16:14
Co-Authored-By: Seluj78 <jlasne@student.42.fr>
@Seluj78
Copy link
Collaborator Author

Seluj78 commented Nov 29, 2018

@JulienPalard les suggestions qui sont fermées auto de @awecx sont peut etre fermées auto par GH quand je push qq chose qui modifie la ligne dont le commentaire fait référence

awecx and others added 2 commits November 29, 2018 16:16
Co-Authored-By: Seluj78 <jlasne@student.42.fr>
Co-Authored-By: Seluj78 <jlasne@student.42.fr>
@JulienPalard
Copy link
Member

@JulienPalard les suggestions qui sont fermées auto de @awecx sont peut etre fermées auto par GH quand je push qq chose qui modifie la ligne dont le commentaire fait référence

Oui, probablement. Mais ça t'arrive régulièrement sur tes PR donc je pense qu'il faut que tu y prête attention.

awecx and others added 3 commits November 30, 2018 14:08
Co-Authored-By: Seluj78 <jlasne@student.42.fr>
Co-Authored-By: Seluj78 <jlasne@student.42.fr>
@Seluj78
Copy link
Collaborator Author

Seluj78 commented Nov 30, 2018

Should be good for another pass on review :)

Co-Authored-By: Seluj78 <jlasne@student.42.fr>
@Seluj78
Copy link
Collaborator Author

Seluj78 commented Dec 11, 2018

Comments from @JulienPalard were fixed.

@JulienPalard JulienPalard merged commit f79fbfa into python:3.7 Dec 13, 2018
@JulienPalard
Copy link
Member

👍

@Seluj78 Seluj78 deleted the using-unix branch December 14, 2018 09:16
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.

3 participants