search

python / python-docs-fr

Mirroir de https://git.afpy.org/AFPy/python-docs-fr
https://docs.python.org/fr/
Other
353 stars 268 forks source link

Library html parser po #1864

Closed ghost closed 2 years ago

ghost commented 2 years ago

Certaines phrases méritent relecture (tournures lourdes et une phrase en quasi mot à mot, par flemme de regarder le contexte au moment de la rédaction). Mais comme c'est globalement fini je publie.

Je viens de me rendre compte qu'il y a des fichiers que je n'ai pas inclus dans mes commits… j'ai encore des choses à comprendre me semble-t-il!

jeanas commented 2 years ago

Au fait, il faudrait aussi passer un coup de powrap pour contenter la CI.

ghost commented 2 years ago

Tu as encore des modifications à logging.po ainsi qu'à whatsnew/3.10.po et un nouveau fichier logging.po.orig :-)

Essaie de faire ça pour corriger :

git switch 3.10
git pull
git switch library-html-parser-po
git rm library/logging.po.orig
git checkout 3.10 -- library/logging.po whatsnew/3.10.po
git add library/logging.po whatsnew/3.10.po
git commit -m "Correction de changements involontaires"
git push

DSl, je ne vois vraiment pas d'où ça peut venir (j'ai même recréer une branche propre avant la PR mais rien n'y fait.

Par contre

(venv) [thierry]>~/D/python-docs-fr:library-html-parser-po$ git switch 3.10                                                            
git : 'switch' n'est pas une commande git. Voir 'git --help'.

Problème avec la version de git?

jeanas commented 2 years ago

Oui, tu sembles utiliser une version de Git légèrement ancienne. La commande git switch a été introduite ainsi que la commande git restore pour éviter la confusion chez les débutants liée au fait qu’on faisait auparavant les deux avec git checkout. Donc, essaie la même chose en remplaçant switch par checkout. Je vois que tu as déjà supprimé logging.po.orig, mais les autres changements involontaires y sont encore. D’ailleurs, est-ce que tu peux aussi copier dans un commentaire le début de ce que te donne

git log 3.10

Je soupçonne que tes problèmes viennent du fait d’avoir committé sur 3.10.

Le 8 mai 2022 à 08:04, Thierry Pellé @.***> a écrit :

 Tu as encore des modifications à logging.po ainsi qu'à whatsnew/3.10.po et un nouveau fichier logging.po.orig :-)

Essaie de faire ça pour corriger :

git switch 3.10 git pull git switch library-html-parser-po git rm library/logging.po.orig git checkout 3.10 -- library/logging.po whatsnew/3.10.po git add library/logging.po whatsnew/3.10.po git commit -m "Correction de changements involontaires" git push DSl, je ne vois vraiment pas d'où ça peut venir (j'ai même recréer une branche propre avant la PR mais rien n'y fait.

Par contre

(venv) [thierry]>~/D/python-docs-fr:library-html-parser-po$ git switch 3.10
git : 'switch' n'est pas une commande git. Voir 'git --help'. Problème avec la version de git?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.

ghost commented 2 years ago

Sortie de git log.

git-log-3.10.txt

Si je comprends c'est pour voir si j'ai pas introduit (involontairement) des modifs dans la branche 3.10 locale, c'est bien ça?

ghost commented 2 years ago

Pour la longueur des lignes j'ai beau appliquer powrap mais ça donne rien il semble.


[...]
rien à valider, la copie de travail est propre
>powrap library/html.parser.po
Fixing wrapping of po files: 100%|█████| 1/1 [00:00<00:00, 18.48it/s]
>git status
[...]
rien à valider, la copie de travail est propre
jeanas commented 2 years ago

Oui, c’est bien ce que je pensais. Dans ton dépôt, il y a une branche 3.10 locale et une branche distante upstream/3.10, synchronisées entre elles. On peut faire des commits sur sa branche principale en local si c’est ensuite pour les pousser ensuite directement sur le dépôt distant (avec ses droits d’écriture), typiquement dans des projets à un développeur. En revanche, ici, et en général quand on utilise GitHub, GitLab ou autre à plusieurs, il ne faut pas le faire, car toutes tes contributions proviennent d’une branche qui est au départ fondée sur 3.10. Résultat, toutes tes PRs ont ces changements. Essaie de faire ceci :

git checkout 3.10

sauvegarde de l’état actuel dans une branche, au cas où

git branch sauvegarde-modifs-involontaires

remet 3.10 à l’état de origin/3.10

git reset --hard origin/3.10

Le 8 mai 2022 à 08:34, Thierry Pellé @.***> a écrit :

 Sortie de git log.

git-log-3.10.txt

Si je comprends c'est pour voir si j'ai pas introduit (involontairement) des modifs dans la branche 3.10 locale, c'est bien ça?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.

jeanas commented 2 years ago

Je vois que tu as un commit

Merge branch 'library-html-parser-po' of github.com:th71/python-docs-fr into library-html-parser-po

Ce n'est pas bon :-) Je parie que tu as fait des modifications aux commits existants (par exemple git commit --amend), et git a râlé au moment du git push, et que tu as suivi son conseil entre parenthèses de faire git pull. Git refuse le push pour éviter que tu n'écrases du travail poussé par d'autres dans l'intervalle. En l'occurrence, ce n'est pas le cas : tu souhaites bel et bien écraser l'ancienne version, et pas la fusionner avec la nouvelle. Il aurait plutôt fallu faire git push --force-with-lease.

Est-ce que tu veux que je m'occupe de te refaire une branche propre ?

ghost commented 2 years ago

C'est pas gagné… Merci pour ta patience. Heu non il ne m'a pas semblé faire ce que tu as fait. Si tu peux le faire pour moi et me transmettre copie des commandes pour que j'essaye de comprendre c'est sympa. Ou je peux le faire mais il faudra que tu m'expliques pas à pas comme à un enfant :-/

jeanas commented 2 years ago

Voilà, c'est fait. J'ai exécuté les commandes

# récupère ta branche depuis ton fork
git fetch git@github.com:th71/python-docs-fr.git library-html-parser-po
# en fait une branche locale chez moi
git checkout -b library-html-parser-po FETCH_HEAD
# garde les fichiers tels quels mais revient à l'état de l'historique de 3.10
git reset 3.10
# nouveau commit, avec --author pour te l'attribuer
git commit --author="Thierry Pellé <th71@users.noreply.github.com>" library/html.parser.po
# suppression de tous les changements autres que sur library/html.parser.po
git reset --hard HEAD
# et mise à jour de la PR
git push git@github.com:th71/python-docs-fr HEAD:library-html-parser-po --force
jeanas commented 2 years ago

Ah non, tu viens de faire git pull. Ce qui détruit mon travail :-) Désolé, j'aurais dû y penser. À la place, ce coup-ci, il faut faire

git fetch origin
git reset --hard origin/library-html-parser-po

Ne touche à rien, je refais ce que j'ai fait tout à l'heure.

ghost commented 2 years ago

DSL :-< J'attend ton feu vert pour faire ma prochaine c***

jeanas commented 2 years ago

OK, ça devrait être bon. Maintenant, fais

git fetch origin
git reset --hard origin/library-html-parser-po

et vois ce que te dit git log. Normalement, le premier commit devrait être 1fe511c1e15d6a3de370ddca8a9e7fbfb138a937. Si c'est bien le cas, tu peux faire les modifs normalement.

Et ne t'inquiètes pas, beaucoup de gens (y compris moi) passent par des débuts difficiles avec Git.

jeanas commented 2 years ago

D'ailleurs, cette opération a calmé la CI. Le problème résidait dans le fait que tu avais d'autres changements que ce fichier dans la PR, qui eux ne respectaient pas la longueur des lignes.

ghost commented 2 years ago

Yep, git log retourne bien 1fe511c1e15d6a3de370ddca8a9e7fbfb138a937.

Mais je pense que je commence à comprendre; quand j'ai vu les commandes à faire

git fetch origin
git reset --hard origin/library-html-parser-po

je me suis rappellé (en fait je le sais mais je te l'ai pas mentionné) en fait l'origin en provenance de github s'appelle chez moi github. origin est en fait le dépôt Framagit que j'utilisais avant de basculer sur Github pour pouvoir fare des PR... Et donc j'ai remplacé

git fetch origin
git reset --hard origin/library-html-parser-po

par 

git fetch github
git reset --hard github/library-html-parser-po

C'est le dépôt Framagit qui doit foutre la m****! Je vais le détruire et renommer github en origin ;-)

Et ne t'inquiètes pas, beaucoup de gens (y compris moi) passent par des débuts difficiles avec Git.

Et si en plus j'en rajoute en faisant des trucs non standards… Mais c'est vrai que j'ai du mal à voir avant d'entrer une commande ce qu'elle va faire…

Merci en tout cas.

jeanas commented 2 years ago

Ah oui. Tu peux le faire avec

git remote remove origin
git remote rename github origin
ghost commented 2 years ago 

git remote rename github origin```

C'est exactement ce que j'ai fait :-) Finalement mon cas n'est pas si désespéré ;-)

jeanas commented 2 years ago

Tout m'a l'air bon, j'attends que la CI passe pour merger.

PyDocTeur commented 2 years ago

Hello @th71 ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter. Merci de le corriger ou d'ajouter le label meta si c'est une PR spéciale. Un exemple de titre valide serait : « Traduction de dossier/fichier.po ».

Disclaimer Je suis un robot fait par l'équipe de [l'AFPy et de Traduction](https://github.com/AFPy/PyDocTeur/graphs/contributors) sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs. [Code source](https://github.com/afpy/pydocteur) I'm a bot made by the [Translation and AFPy teams](https://github.com/AFPy/PyDocTeur/graphs/contributors) on their free time. I might say or do dumb things sometimes. Don't blame me, blame the developer ! [Source code](https://github.com/afpy/pydocteur) (state: incorrect_title) `PyDocTeur v1.12.0`
ghost commented 2 years ago

Cool! Merci à toi.

Ah ben non. Effectivement j'aurais pas du mettre -po...

Bonne fin de WE!