8000 Library html parser po · Pull Request #1864 · python/python-docs-fr · GitHub
[go: up one dir, main page]

Skip to content

Library html parser po #1864

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.

8000

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from May 8, 2022
Merged

Library html parser po #1864

merged 5 commits into from May 8, 2022

Conversation

ghost
Copy link
@ghost ghost commented May 7, 2022

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!

Copy link
Collaborator
@jeanas jeanas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@jeanas
Copy link
Collaborator
jeanas commented May 7, 2022

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

@ghost
Copy link
Author
ghost commented May 8, 2022

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
Copy link
Collaborator
jeanas commented May 8, 2022 via email

Copy link
Author
@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Première salve de modifications.

@ghost ghost closed this May 8, 2022
@ghost ghost reopened this May 8, 2022
@ghost
Copy link
Author
ghost commented May 8, 2022

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
Copy link
Author
ghost commented May 8, 2022

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
Copy link
Collaborator
jeanas commented May 8, 2022 via email

@jeanas
Copy link
Collaborator
jeanas commented May 8, 2022

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 g 8000 it 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
Copy link
Author
ghost commented May 8, 2022

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
Copy link
Collaborator
jeanas commented May 8, 2022

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
Copy link
Collaborator
jeanas commented May 8, 2022

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
Copy link
Author
ghost commented May 8, 2022

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

@jeanas
Copy link
Collaborator
jeanas commented May 8, 2022

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
Copy link
Collaborator
jeanas commented May 8, 2022

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
Copy link
Author
ghost commented May 8, 2022

Yep,
git log retourne bien 1fe511c.

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
Copy link
Collaborator
jeanas commented May 8, 2022

Ah oui. Tu peux le faire avec

git remote remove origin
git remote rename github origin

@ghost
Copy link
Author
ghost commented May 8, 2022
git remote rename github origin```

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

@jeanas
Copy link
Collaborator
jeanas commented May 8, 2022

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

@PyDocTeur
Copy link

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
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: incorrect_title)
PyDocTeur v1.12.0

@jeanas jeanas merged commit c4cc2a5 into python:3.10 May 8, 2022
@ghost
Copy link
Author
ghost commented May 8, 2022

Cool!
Merci à toi.

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

Bonne fin de WE!

@ghost ghost deleted the library-html-parser-po branch May 8, 2022 08:35
@jeanas jeanas mentioned this pull request May 16, 2022
JulienPalard pushed a commit that referenced this pull request May 22, 2022
Co-Authored-By: Jean Abou Samra <jean@abou-samra.fr>
JulienPalard pushed a commit that referenced this pull request May 22, 2022
Co-Authored-By: Jean Abou Samra <jean@abou-samra.fr>
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
0