Skip to content

Dict #911

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.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Dict #911

wants to merge 1 commit into from

Conversation

hucste
Copy link
Contributor

@hucste hucste commented Oct 29, 2019

ajout terme bzip2 & multiflux

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 5, 2019

Oula c'est une grosse PR qui viens d'une vielle branche. Est tu sur ? :) Et est ce que tu pourrais résoudre les conflits de merge avant que on commence a review?

@hucste
Copy link
Contributor Author

hucste commented Nov 7, 2019

oulahhh, je ne pensais pas à "mal"… :p
Il vaut peut être mieux que je supprime cette requête, et que je reparte sur un upstream/3.8, non ?!

Parce que là, gérer ça. moi en mode un peu peur de faire des bêtises, tel que tout "casser", alors que je n'ai jamais fait ça, et que je ne sais pas comment réparer si jamais il y a besoin !

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 15, 2019

@hucste yes je pense aussi ! je te laisse close ?

@deronnax
Copy link
Collaborator

Sinon, autre solution devrait pas être trop dur : tu fais git rebase -i 3.8, et tu retires de la liste les commit qui ne sont pas les tiens. Ensuite git push --force. Ça devrait régler le problème. Si tu es un peu à l'aise avec git, ça le fera. Et ça sera moins de travail que de refaire une PR

@hucste
Copy link
Contributor Author

hucste commented Nov 16, 2019

Sinon, autre solution devrait pas être trop dur : tu fais git rebase -i 3.8, et tu retires de la liste les commit qui ne sont pas les tiens. Ensuite git push --force. Ça devrait régler le problème. Si tu es un peu à l'aise avec git, ça le fera. Et ça sera moins de travail que de refaire une PR

Mais alors, pas du tout à l'aise, mais vraiment pas du tout. Si je comprends le principe de ce que tu me décris @deronnax, je ne sais pas et ne vois pas comment faire, surtout la partie "retirer de la liste les commits qui ne sont pas miens" !

Désolé.

Donc selon ta réponse, @deronnax j'agirais en conséquence. Soit fermeture, soit suivis des instructions développées.

Voili, voilou.

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 16, 2019

Alors je te fait une explication avec mes mots, peut être que @deronnax fera mieux :)

Tu va dans ton terminal, dans le dossier python-docs-fr avec la bonne branche séléctionnée. Ensuite tu lance la commande git rebase -i 3.8

le -i va faire un rebase interactif, ce qui va te permettre de choisir quels commits sont conservés.

L'idée est donc que quand tu lance cette commande, un editeur genre vim ou emacs ou nano va s'ouvrir et tu va voir une liste de commits. Il te suffit de supprimer tous les commits qui ne sont pas les tiens (cad ceux des autres). Pour les supprimer, il suffit de supprimer la ligne.

Une fois fini, ferme le fichier en le sauvegardant bien entendu, puis git push --force et ca devrait etre bon

@hucste
Copy link
Contributor Author

hucste commented Nov 16, 2019

Ok, merci pour les explications. en effet, cela a l'air très simple. Je vais m'y atteler maintenant ! :p

@hucste
Copy link
Contributor Author

hucste commented Nov 16, 2019

Tu va dans ton terminal, dans le dossier python-docs-fr avec la bonne branche séléctionnée. Ensuite tu lance la commande git rebase -i 3.8

Ahhh, pas de chance !
Une fois dans la branche dict, il ne connaît pas 3.8

$ git co dict
Basculement sur la branche 'dict'
Votre branche est à jour avec 'origin/dict'.

$ git rebase -i 3.8
fatal: amont invalide '3.8'

???

git rebase -i upstream/3.8, plutôt non ?

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 16, 2019

yes, fait un git fetch upstream d'abord

@hucste
Copy link
Contributor Author

hucste commented Nov 17, 2019

yes, fait un git fetch upstream d'abord

Non, c'est pareil, même soucis !

(sachant que git co est un alias perso pour git checkout)

$ git fetch upstream
remote: Enumerating objects: 5, done.
remote: Counting objects: 100% (5/5), done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 5 (delta 0), reused 2 (delta 0), pack-reused 0
Dépaquetage des objets: 100% (5/5), fait.
Depuis https://github.com/python/python-docs-fr
   9fef0f69..ef3cbeb5  3.7                       -> upstream/3.7
 * [nouvelle branche]  revert-619-asyncio-policy -> upstream/revert-619-asyncio-policy

$ git rebase -i 3.8  
fatal: amont invalide '3.8'

$ git co origin
Note : basculement sur 'origin'.

Vous êtes dans l'état « HEAD détachée ». Vous pouvez visiter, faire des modifications
expérimentales et les valider. Il vous suffit de faire un autre basculement pour
abandonner les commits que vous faites dans cet état sans impacter les autres branches

Si vous voulez créer une nouvelle branche pour conserver les commits que vous créez,
il vous suffit d'utiliser l'option -c de la commande switch comme ceci :

  git switch -c <nom-de-la-nouvelle-branche>

Ou annuler cette opération avec :

  git switch -

Désactivez ce conseil en renseignant la variable de configuration advice.detachedHead à false

HEAD est maintenant sur 7580eaf0 Merge pull request #790 from Seluj78/c-api-datetime

$ git fetch upstream

$ git co dict        
La position précédente de HEAD était sur 7580eaf0 Merge pull request #790 from Seluj78/c-api-datetime
Basculement sur la branche 'dict'
Votre branche est à jour avec 'origin/dict'.

$ git rebase -i 3.8  
fatal: amont invalide '3.8'

:(

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 26, 2019

Je bosse sur un rebase

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 26, 2019

Fixed 😎

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 26, 2019

@hucste Voici comment j'ai fait:

➜  hucte git:(dict) git remote add upstream https://github.com/python/python-docs-fr.git
Alias tip: gra upstream https://github.com/python/python-docs-fr.git
➜  hucte git:(dict) git remote -v
Alias tip: grv
origin	git@github.com:hucste/python-docs-fr.git (fetch)
origin	git@github.com:hucste/python-docs-fr.git (push)
upstream	https://github.com/python/python-docs-fr.git (fetch)
upstream	https://github.com/python/python-docs-fr.git (push)
➜  hucte git:(dict) git fetch
Alias tip: gf
➜  hucte git:(dict) git fetch upstream
Alias tip: gf upstream
remote: Enumerating objects: 636, done.
remote: Counting objects: 100% (636/636), done.
remote: Compressing objects: 100% (10/10), done.
remote: Total 877 (delta 626), reused 622 (delta 622), pack-reused 241
Receiving objects: 100% (877/877), 1.10 MiB | 1.35 MiB/s, done.
Resolving deltas: 100% (749/749), completed with 340 local objects.
From https://github.com/python/python-docs-fr
 * [new branch]        2.7                            -> upstream/2.7
 * [new branch]        3.5                            -> upstream/3.5
 * [new branch]        3.6                            -> upstream/3.6
 * [new branch]        3.7                            -> upstream/3.7
 * [new branch]        3.8                            -> upstream/3.8
 * [new branch]        JulienPalard-issue-templates   -> upstream/JulienPalard-issue-templates
 * [new branch]        master                         -> upstream/master
 * [new branch]        readme-glossaire-whitespace    -> upstream/readme-glossaire-whitespace
 * [new branch]        revert-619-asyncio-policy      -> upstream/revert-619-asyncio-policy
 * [new branch]        revert-899-using-cmdline       -> upstream/revert-899-using-cmdline
 * [new branch]        revert-984-travis_commit_range -> upstream/revert-984-travis_commit_range
➜  hucte git:(dict) git rebase -i upstream/3.8
Alias tip: grbi upstream/3.8
Successfully rebased and updated refs/heads/dict.
➜  hucte git:(dict) git push --force
Alias tip: gpf!
Enumerating objects: 5, done.
Counting objects: 100% (5/5), done.
Delta compression using up to 12 threads
Compressing objects: 100% (3/3), done.
Writing objects: 100% (3/3), 367 bytes | 367.00 KiB/s, done.
Total 3 (delta 2), reused 0 (delta 0)
remote: Resolving deltas: 100% (2/2), completed with 2 local objects.
To github.com:hucste/python-docs-fr.git
 + 1b22b0ef...636a8245 dict -> dict (forced update)
➜  hucte git:(dict)

@deronnax
Copy link
Collaborator

@hucste ça aurait eu sa place dans ta PR pour bzip2 :)

@hucste
Copy link
Contributor Author

hucste commented Nov 26, 2019

@hucste ça aurait eu sa place dans ta PR pour bzip2 :)

Oui, je crois qu'à vouloir trop bien faire les choses, je les ai compliquées pour tout le monde.

@Seluj78 : merci 👍 :D

Donc, de mon côté, je fais quoi ?
je détruis ma branche localement ? et celle sur mon dépôt ici ?!

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 26, 2019

@hucste non tu as juste a attendre que on valide ou non cette PR :)

@JulienPalard
Copy link
Member

Cette PR me semble un peu vide, si un jour on a besoin de ces termes, on le commitera dans le même commit que le .po qui l'utilise.

@deronnax
Copy link
Collaborator

deronnax commented Dec 6, 2019

Je lui ai déjà dit :) : #911 (comment)

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.

4 participants