-
Notifications
You must be signed in to change notification settings - Fork 266
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.
8000Already on GitHub? Sign in to your account
Library html parser po #1864
Conversation
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.
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
Au fait, il faudrait aussi passer un coup de |
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
Problème avec la version de git? |
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.
|
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.
Première salve de modifications.
Sortie de git log. 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? |
Pour la longueur des lignes j'ai beau appliquer powrap mais ça donne rien il semble.
|
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.
|
Je vois que tu as un commit
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 Est-ce que tu veux que je m'occupe de te refaire une branche propre ? |
C'est pas gagné… Merci pour ta patience. |
Voilà, c'est fait. J'ai exécuté les commandes
|
Ah non, tu viens de faire
Ne touche à rien, je refais ce que j'ai fait tout à l'heure. |
DSL :-< |
Co-Authored-By: Jean Abou Samra <jean@abou-samra.fr>
OK, ça devrait être bon. Maintenant, fais
et vois ce que te dit Et ne t'inquiètes pas, beaucoup de gens (y compris moi) passent par des débuts difficiles avec Git. |
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. |
Yep, Mais je pense que je commence à comprendre; quand j'ai vu les commandes à faire
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.
par
C'est le dépôt Framagit qui doit foutre la m****!
Et si en plus j'en rajoute en faisant des trucs non standards… Merci en tout cas. |
Ah oui. Tu peux le faire avec
|
|
Tout m'a l'air bon, j'attends que la CI passe pour merger. |
Hello @th71 ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter. DisclaimerJe suis un robot fait par l'équipe de l'AFPy et de Traduction I'm a bot made by the Translation and AFPy teams on their free (state: incorrect_title) |
Cool! Ah ben non. Effectivement j'aurais pas du mettre -po... Bonne fin de WE! |
Co-Authored-By: Jean Abou Samra <jean@abou-samra.fr>
Co-Authored-By: Jean Abou Samra <jean@abou-samra.fr>
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!