Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Merged
merged 44 commits into from
Nov 27, 2021
Merged

Conversation

ygingras
Copy link
Contributor

Première partie: #1761
Il y aura une 3e partie finale après celle ci.

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 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 ?

@ygingras
Copy link
Contributor Author

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é?

@JulienPalard
Copy link
Member

JulienPalard commented Nov 23, 2021

Wah. Ça mérite probablement une modif du Makefile :

EXCLUDED := \
	whatsnew/ \
	c-api/ \
	distutils/ \
        ...

ce qui fait que le make todo ne voit pas ton travail ni les fuzzy qui pourraient y survenir. Il ne le comptabilise pas non plus dans le travail fait.

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 :

$ make todo
[...]

# whatsnew (79.86% done)

- 3.10.po                        339 / 496 ( 68.0% translated)
- 3.9.po                         348 / 365 ( 95.0% translated)

@JulienPalard
Copy link
Member

(relu jusqu'a la ligne 1841)

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 grande contribution.
Relu jusqu'à la ligne 2177.

…lard

Co-authored-by: Christophe Nanteuil <[email protected]>
Co-authored-by: Julien Palard <[email protected]>
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 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.

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.

super boulot, merci !

@christopheNan christopheNan merged commit 99ffbb6 into python:3.10 Nov 27, 2021
@ygingras
Copy link
Contributor Author

Super! Je soumets la partie finale d'ici quelques jours.

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