Skip to content

Conversation

@drNymbus
Copy link
Contributor

No description provided.

Copy link

@awecx awecx 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 PR.
Tu peux aplliquer les changements directement depuis Github mais un passage de powrap sera sans doute nécessaire.
Il te faudra récupérer les changements fait depuis Github en pullant ces modifications sur ton repo local, passer powrap et re-submitter le résulatt.

Copy link

@awecx awecx 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 PR.
Tu peux appliquer les changements directement depuis Github mais un passage de powrap sera sans doute nécessaire.
Il te faudra récupérer les changements fait depuis Github en pullant ces modifications sur ton repo local, passer powrap et re-submitter le résulatt.

@deronnax
Copy link
Collaborator

deronnax commented Nov 4, 2019

en langage de terminal :
git pull --rebase, powrap puis git commit

drNymbus and others added 17 commits November 7, 2019 13:47
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 7, 2019

@drNymbus tu peux utiliser des patchs plutot que de faire un par un commit ! :)

@Seluj78 Seluj78 changed the title Advancement in whatsnew/2.0.po, 1 fuzzy created for the word "workhorse" Advancement in whatsnew/2.0.po Nov 7, 2019
@deronnax
Copy link
Collaborator

deronnax commented Nov 7, 2019

@drNymbus tu peux utiliser des patchs plutot que de faire un par un commit ! :)

Des patchs ? Tu veux dire des batchs, non ?
@drNymbus dans la vue "conversation", tu ne peux qu'ajouter les suggestions une par une. Mais si tu vas dans la vue "file changed", tu as un autre bouton dans les suggestions qui est "add to batch", et ça te créé un "lot" de suggestions à ajouter, et à la fin tu as un bouton genre "commit batch", et ça ajoute toutes les modifications en un seul commit ;)

@vpoulailleau
Copy link
Contributor

@drNymbus tu peux utiliser des patchs plutot que de faire un par un commit ! :)

Des patchs ? Tu veux dire des batchs, non ?
@drNymbus dans la vue "conversation", tu ne peux qu'ajouter les suggestions une par une. Mais si tu vas dans la vue "file changed", tu as un autre bouton dans les suggestions qui est "add to batch", et ça te créé un "lot" de suggestions à ajouter, et à la fin tu as un bouton genre "commit batch", et ça ajoute toutes les modifications en un seul commit ;)

Je ne connaissais pas, et c'est vachement bien. Ça mériterait d'être mentionné dans le README, non ?

@deronnax
Copy link
Collaborator

deronnax commented Nov 7, 2019

Beuh, je suis pas trop chaud pour deux raisons :

  • c'est du Github standard, c'est dans la doc Github
  • plus le README est gros moins les gens le lisent (surtout les jeunes(c'est mon constat perso)), et les gens ne le lisent déjà pas (donc l'ajouter ne réglera pas le problème)

Après je suis pas vent debout contre non plus et on peut intégrer le lien de la doc github de la fonctionnalité si tout le monde le souhaite

@vpoulailleau
Copy link
Contributor

* plus le README est gros moins les gens le lisent

Oui, je partage ton point de vue. Je me faisais la réflexion tout à l'heure qu'il pourrait y avoir un README résumé (le README principal), et une version détaillée pour quand quelqu'un butte sur un sujet. Vous (l'équipe python-docs-fr) avez plus l'habitude que moi pour les interactions avec la communauté de traduction, mais j'imagine que c'est toujours un peu les mêmes questions qui reviennent, d'où mon idée de README détaillé, que l'on utiliserait pour dire la réponse à ta question se trouve là, nous pourrons la détailler si elle n'est pas claire.

Et globalement, à ce que j'ai pu voir pendant la PyConFr, les gens n'utilisaient pas trop le README, car il y avait quelqu'un pour expliquer. Mais quand on n'est pas en présentiel, il ne faut pas que les gens soient bloqués s'ils ne trouvent pas l'info.

Voilà pour mes suggestions ;-) Je vous laisse prendre la décision qui convient le mieux à l'équipe python-docs-fr.

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 15, 2019

Hello @drNymbus ! Petit ping ! 🏓

@Seluj78 Seluj78 requested a review from awecx December 11, 2019 15:02
Copy link

@awecx awecx left a comment

Choose a reason for hiding this comment

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

Salut @drNymbus.
Tu pourras powraper (réindenter) ton fichier ?
On pourra merger après.

@Seluj78
Copy link
Collaborator

Seluj78 commented Feb 4, 2020

Ping @drNymbus

@deronnax
Copy link
Collaborator

Hello @drNymbus . Tu te sens d'intégrer les derniers changements restants ;) ?

@deronnax
Copy link
Collaborator

je la reprends

@deronnax deronnax assigned deronnax and unassigned drNymbus and awecx Mar 29, 2020
@deronnax deronnax requested review from awecx and removed request for awecx May 2, 2020 20:58
@deronnax deronnax requested a review from awecx May 2, 2020 20:58
@deronnax
Copy link
Collaborator

deronnax commented May 2, 2020

@awecx il faut que tu approuves la PR

@deronnax deronnax merged commit 3d088ca into python:3.8 May 2, 2020
@deronnax
Copy link
Collaborator

deronnax commented May 2, 2020

ENFIN !

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.

5 participants