Skip to content

Ajout d'une section sur la gestion des suggestions #1642

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

Merged
merged 3 commits into from
May 24, 2021

Conversation

dmerejkowsky
Copy link
Contributor

Un ajout dans CONTRIBUTING.rst que je trouve utile.

Peut-être qu'il faut rajouter une section pour expliquer comment appliquer toutes les suggestions en un seul commit
(l'astuce c'est de passer par la vue "Changed files" et de cliquer sur "Add suggestion to batch")

@christopheNan
Copy link
Contributor

Pertinent et utile dans d'autres cas aussi. Je pense que tu peux reformuler en disant « si vous avez besoin de reprendre votre PR sur votre ordinateur après avoir fait des modifications en ligne sur github, par exemple si … ».
Je pense qu'il manque qu'il manque un git add <fichier.po> avant le git commit

@python python deleted a comment from PyDocTeur May 20, 2021
@dmerejkowsky
Copy link
Contributor Author

J'ai rajouté le git add qui manquait, bien vu.

si vous avez besoin de reprendre votre PR sur votre ordinateur après avoir fait des modifications en ligne sur github, par exemple si

Cette formulation est plus correcte mais je la trouve un peu lourde - je pense qu'on peut trouver mieux

christopheNan
christopheNan previously approved these changes May 24, 2021
@PyDocTeur
Copy link

ON Y EST PRESQUE ! Un p'tit label automerge et je merge ça !


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: approved)
PyDocTeur v1.12.0

Copy link
Member

@JulienPalard JulienPalard left a comment

Choose a reason for hiding this comment

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

Oui super idée d'ajout !

CONTRIBUTING.rst Outdated
@@ -233,6 +233,18 @@ Mettez dans le commentaire de la *pull request* le texte suivant :
« Closes #XXXX » où XXXX est le numéro du ticket GitHub créé pour réserver le fichier traduit.
Cela permet à Github de lier la *pull request* au ticket de réservation.

Il est possible que GitHub vous laisse la possibilité de faire un commit
automatique contenant les suggestions proposées pendant la revue. Cela fonctionne bien, mais
parfois cela casse l'indentation. Si cela arrive, vous pouvez récupérer le commit fait par GitHUb
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
parfois cela casse l'indentation. Si cela arrive, vous pouvez récupérer le commit fait par GitHUb
le résultat n'est pas toujours convenable pour ``powrap``. Si cela arrive, vous pouvez récupérer le commit fait par GitHub

Il n'y a pas d'indentation ici, plutôt du wrapping. (Moi aussi je fais l'erreur, powrap s'appelait poindent avant :D)

CONTRIBUTING.rst Outdated
Il est possible que GitHub vous laisse la possibilité de faire un commit
automatique contenant les suggestions proposées pendant la revue. Cela fonctionne bien, mais
parfois cela casse l'indentation. Si cela arrive, vous pouvez récupérer le commit fait par GitHUb
puis ré-indenter les fichiers de traduction :
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
puis ré-indenter les fichiers de traduction :
puis relancer ``powrap`` :

@JulienPalard JulienPalard merged commit 60ed982 into python:3.9 May 24, 2021
@JulienPalard
Copy link
Member

Merci @dmerejkowsky !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants