-
Notifications
You must be signed in to change notification settings - Fork 266
Seconde partie de whatsnew/3.10 #1777
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.
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
Conversation
Co-authored-by: Jean-Abou-Samra <[email protected]>
Co-authored-by: Jean-Abou-Samra <[email protected]>
…whatsnew-310-2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 <[email protected]>
Co-authored-by: Christophe Nanteuil <[email protected]>
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) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci pour cette grande contribution.
Relu jusqu'à la ligne 2177.
…lard Co-authored-by: Christophe Nanteuil <[email protected]> Co-authored-by: Julien Palard <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 <[email protected]>
Co-authored-by: Christophe Nanteuil <[email protected]>
Co-authored-by: Christophe Nanteuil <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.