Skip to content

Seconde partie de whatsnew/3.10#1777

Merged
christopheNan merged 44 commits into
python:3.10from
ygingras:whatsnew-310-2
Nov 27, 2021
Merged

Seconde partie de whatsnew/3.10#1777
christopheNan merged 44 commits into
python:3.10from
ygingras:whatsnew-310-2

Conversation

@ygingras

Copy link
Copy Markdown
Contributor

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

@christopheNan christopheNan left a comment

Copy link
Copy Markdown
Contributor

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 ?

Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
@ygingras

Copy link
Copy Markdown
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

JulienPalard commented Nov 23, 2021

Copy link
Copy Markdown
Member

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)

Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
@JulienPalard

Copy link
Copy Markdown
Member

(relu jusqu'a la ligne 1841)

@christopheNan christopheNan left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread Makefile
…lard

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-authored-by: Julien Palard <julien@palard.fr>

@christopheNan christopheNan left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
Comment thread whatsnew/3.10.po Outdated
ygingras and others added 9 commits November 24, 2021 15:46
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 christopheNan left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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