[library/wsgiref.po] initial partial translation#900
Conversation
Co-Authored-By: Jules Lasne (jlasne) <jlasne@student.42.fr>
Co-Authored-By: Jules Lasne (jlasne) <jlasne@student.42.fr>
Co-Authored-By: Jules Lasne (jlasne) <jlasne@student.42.fr>
Co-Authored-By: Jules Lasne (jlasne) <jlasne@student.42.fr>
Co-Authored-By: Jules Lasne (jlasne) <jlasne@student.42.fr>
awecx
left a comment
There was a problem hiding this comment.
Merci pour cette PR.
Je t'ai noté quelques points qui méritent relecture.
awecx
left a comment
There was a problem hiding this comment.
Merci pour cette PR.
Je t'ai noté quelques points qui méritent relecture.
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
|
@awecx énorme merci pour cette relecture et les suggestions de corrections ! |
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
|
Il reste qq points de détails pour pouvoir merger, le « Renvoie une heuristique » et ligne 176—179. |
|
Quelques souci d'orthographe et de wrapping selon travis: |
Co-Authored-By: Julien Palard <julien@palard.fr>
|
Bonjour, |
|
@arthurlogilab ping ! :) 🏓 |
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
awecx
left a comment
There was a problem hiding this comment.
On y est presque, il reste quelques erreurs remontées par pospell.
Je te les ai mises en suggestion , on pourra soumettre ta PR ensuite.
Merci
vpoulailleau
left a comment
There was a problem hiding this comment.
Et quelques petites corrections suggérées par padpo.
padpo -p 900
Co-Authored-By: Mathieu Dupuy <deronnax@users.noreply.github.com> Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com> Co-Authored-By: Vincent Poulailleau <vpoulailleau@gmail.com>
Co-Authored-By: Vincent Poulailleau <vpoulailleau@gmail.com>
awecx
left a comment
There was a problem hiding this comment.
Je ne sais pas écrire « environnement »...
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
awecx
left a comment
There was a problem hiding this comment.
Pas de chance, ça a cassé le wrapping.
Relance powrap sur le code et repush, c'est presque bon.
There was a problem hiding this comment.
Je n'avais pas vu, tu n'as pas pris en compte la remarque de @deronnax :
« et toutes les autres variables WSGI définies dans »
Il te faut soit l'accepter (attention à la taille de la ligne), soit tu résous la conversation avec un commentaire pour expliquer ton choix.
On y est vraiment presque là.
awecx
left a comment
There was a problem hiding this comment.
Deux remarques qui ont dû sauter entre deux commits.
|
@arthurlogilab on y est presque. Tu peux voir mes remarques et celle de deronnax ? |
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
christopheNan
left a comment
There was a problem hiding this comment.
Bon pour merge en ce qui me concerne.
|
Merci pour cette PR ! |
No description provided.