Skip to content

Conversation

@Catadanna
Copy link
Contributor

No description provided.

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 très bonne contribution.

Catadanna and others added 5 commits March 24, 2020 08:25
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>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@Catadanna
Copy link
Contributor Author

Catadanna commented Mar 24, 2020

Juste une question: quand on travaille sur un fichier, est-ce qu'on doit réviser aussi les lignes qui ont été déjà traduites? Car la plupart de tes corrections concernent des phrases que je n'avais pas touchées. Juste pour savoir pour la prochaine fois. Merci.

Catadanna and others added 2 commits March 24, 2020 08:32
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@Catadanna
Copy link
Contributor Author

Est-ce que je dois faire quelque chose maintenant et je dois attendre ? Je m'imagine que les changements que je viens d'approuver ont été commités directement.

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.

Je propose que tu rajoutes bytecode --> « code intermédiaire » dans le glossaire de contributing.rst (éventuellement tuple --> « n-uplet »).
Travis te signale que, suite aux modifications faites via Github, le fichier n'est plus correctement limité à 79 caractères par ligne : « would rewrap ». Tu dois récupérer les modifications faites sur Github (après revue des présentes remarques) via un git pull origin puis ré-aligner le fichier avec powrap ou poedit, git commit et git push.
Tu peux bien évidemment proposer des améliorations aux traductions déjà effectuées, même s'il n'y a pas de marqueur fuzzy.

Catadanna and others added 4 commits March 25, 2020 07:59
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>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@Catadanna
Copy link
Contributor Author

Catadanna commented Mar 25, 2020

J'ai basculé sur webchat, pour ne pas charger ici les messages avec mes questions.

@Catadanna
Copy link
Contributor Author

Catadanna commented Mar 25, 2020

J'ai fait le pull sur origin, refait le powrap sur le fichier en travaillant sur la même branche, et commité de nouveau, et push sur la branche. Il me semble qu'il n'y a pas d'amélioration ...

J'ai regardé les logs de Travis, ils indiquent le contexte de l'erreur, mais quand je le cherche dans poedit je ne le trouve pas (Edition/Recherche). Il n'y a pas de numéro de ligne en poedit? Il y a un ID mais ça ne correspond à aucun indicateur en Travis.

Pourtant je travaille toujours sur la même branche, la preuve est que je vois ici les commits que je fais (le message).

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.

CPython est dans le dictionnaire additionnel. Il n'y a pas besoin de le mettre entre astérisques. Pourquoi le faire ?
Pour les majuscules manquantes en début de phrase, merci de vérifier avant dans le rendu html que c'est bien un oubli et pas volontaire.
J'ai fait le wrapping pour que Travis ne râle pas.

Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Catadanna and others added 5 commits March 27, 2020 07:17
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>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@Catadanna
Copy link
Contributor Author

Travis tourne depuis 10 heures je crois, c'est normal ?

@Catadanna
Copy link
Contributor Author

Révisions prises en compte, fichier traduit entièrement. C'est bon de mon côté.

@Catadanna
Copy link
Contributor Author

J'ai changé en attendables. Ca va bloquer probablement au niveau de Travis, je ne sais pas s'il recconaît le mot.

@Catadanna
Copy link
Contributor Author

Catadanna commented Apr 7, 2020

OK fait. Il me semble qu'il y a un souci avec un autre fichier sur la même branche.

@christopheNan
Copy link
Contributor

Oui, j'y suis allé un peu fort en te demandant de rapatrier tout upstream. Corrigeons cela :

  • sur ta branche, récupère l'identifiant du dernier commit 'Cr' avec git log. Tu dois avoir '4f1cca8620…'
  • reviens à l'état avant ce commit : git revert 4f1cca8620. Cela va générer un nouveau commit.
  • plutôt que rapatrier tout upstream, ne prenons que le 'Makefile' : git checkout upstream/3.8 Makefile. Tu devrais pouvoir valider ce commit sans erreur.
@Catadanna
Copy link
Contributor Author

Je me demande si le problème ne vienne pas d'une erreur que j'avais faite au tout début, je crois que j'ai travaillé sur deux fichiers, tout étant sur la mêmle branche notamment reference-datamodel. Il s'agit des deux fichiers : reference/datamodel.po et using/windows.po. De plus j'étais sur la mauvaise version (3.7). Je ne sais pas comment résoudre ça, bien que je fais attention de faire git add et commit seulement pour reference/datamodel.po.

@Catadanna
Copy link
Contributor Author

Catadanna commented Apr 22, 2020

J'ai fait un revert mais j'ai eu cette erreur :

error : le commit 4f1cca8620ec674e3a0fa863ecfd826dc8215142 est une fusion mais l'option -m n'a pas été spécifiée. fatal: revert a échoué 

Je ne vois pas où mettre cet -m car il semble que pour revert il a une autre signification.

J'ai fouillé sur le net un peu je n'ai pas trouvé de solution.

@christopheNan
Copy link
Contributor

Le numéro du commit est un exemple. Sur ton clone local, le numéro sera différent : tu l'obtiens à l'aide de git log.

@Catadanna
Copy link
Contributor Author

Catadanna commented Apr 22, 2020

Mais je sais bien! J'ai mis ton example d'identifiant de commit dans l'erreur, parce que c'était plus simple. (Il est plus long aussi le id). Ca peut te servir d'avoir cet id? Je peux te l'envoyer, voilà j'ai mis dans l'erreur. Ca n'empêche que j'ai cette erreur, je dois associer un commentaire à revert ?

@Catadanna
Copy link
Contributor Author

Catadanna commented Apr 22, 2020

OK, ça c'est ce que j'ai : Quel est l'identifiant ? Celui à côté de commit ou bien celui à côté de merge ? J'avais pris le long à côté de commit.

commit 4f1cca8620ec674e3a0fa863ecfd826dc8215142 (HEAD -> reference-datamodel, origin/reference-datamodel) Merge: 47d79b60 9d4e697c Author: Catadanna <catalina_chircu@yahoo.com> Date: Tue Apr 21 01:49:56 2020 +0200 Cr 
@christopheNan
Copy link
Contributor

Ok, effectivement c'était une fusion et git ne sait pas dire quelle partie de la fusion annuler. Pour trouver quelle branche indiquer derrière -m, il faut :

  • git show --pretty=raw 4f1cca8620e . Il va te lister les identifiants des 2 commits parents aux lignes 3 et 4. Le commit parent n°1 est celui de la ligne 3, le commit parent n°2 est celui de la ligne 4 ;
  • git revert 4f1cca86 -m <numero_du_commit_parent_que_tu_veux_revert> . Toi, tu ne veux pas du commit qui vient de upstream, mais tu veux garder le commit qui vient de ta branche. l'identifiant du commit parent que tu veux garder est donc celui que tu as, dans ta branche, avec git log juste avant le merge.
@Catadanna
Copy link
Contributor Author

Catadanna commented Apr 22, 2020

Donc si je comprends bien, le dernier commit parent est celui de la ligne 3? Puisque j'ai fait le commit Cr après upstream. Qu'est-ce que tu apelles numéro 1 et numéro 2? Quel commit parent est le dernier, ligne 3 ou ligne 4 ?

@christopheNan
Copy link
Contributor

Peux-tu essayer :
git checkout upstream/3.8 using/windows.po

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 ! C'est bon pour moi.

@deronnax deronnax merged commit a4c066c into python:3.8 May 6, 2020
@deronnax deronnax linked an issue May 25, 2020 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants