Seconde partie de whatsnew/3.10#1777
Conversation
Co-authored-by: Jean-Abou-Samra <37271310+Jean-Abou-Samra@users.noreply.github.com>
Co-authored-by: Jean-Abou-Samra <37271310+Jean-Abou-Samra@users.noreply.github.com>
christopheNan
left a comment
There was a problem hiding this comment.
Merci pour cette grosse contribution (revu jusqu'à la ligne 1080). Juste quelques remarques, principalement de la typographie.
Pour le filtrage par motifs et répondre à ta remarque en commentaire, je pense qu'il faut entendre (souvent) motif d'un filtre pour filtre. J'ai donc fait cette précision à quelques endroits, en essayant de ne pas trop alourdir les phrases. Qu'en penses-tu ?
|
Merci pour cette super relecture, @christopheNan ! Je me rends compte que j'ai basé cette PR sur la partie 1 qui n'a pas encore été accepté et la ligne 1067 correspond plus ou mois à la partie 1. Devrais-je fermer ma première PR en continuer le travail dans celle ci ou plutôt intégrer tes commentaire dans la première PR et déplacer la discussion de ce côté? |
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
|
Wah. Ça mérite probablement une modif du Makefile : EXCLUDED := \
whatsnew/ \
c-api/ \
distutils/ \
...ce qui fait que le On peut peut être juste exclure les whatsnew de Python 2, et des quelques vieilles Python 3, la syntaxe est celle de gitignore, donc on peut utiliser : EXCLUDED := \
- whatsnew/ \
+ whatsnew/2.?.po \
+ whatsnew/3.[0-8].po \
c-api/ \Ça donne : |
|
(relu jusqu'a la ligne 1841) |
christopheNan
left a comment
There was a problem hiding this comment.
Merci pour cette grande contribution.
Relu jusqu'à la ligne 2177.
…lard Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> Co-authored-by: Julien Palard <julien@palard.fr>
christopheNan
left a comment
There was a problem hiding this comment.
Merci beaucoup pour cette longue contribution. Le nombre de remarques est juste dû à la longueur de ta contribution et ne reflète pas l'excellente qualité du produit.
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
christopheNan
left a comment
There was a problem hiding this comment.
super boulot, merci !
|
Super! Je soumets la partie finale d'ici quelques jours. |
Première partie: #1761
Il y aura une 3e partie finale après celle ci.