-
Notifications
You must be signed in to change notification settings - Fork 266
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
Conversation
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 … ». |
J'ai rajouté le
Cette formulation est plus correcte mais je la trouve un peu lourde - je pense qu'on peut trouver mieux |
ON Y EST PRESQUE ! Un p'tit label 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: approved) |
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.
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 |
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.
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 : |
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.
puis ré-indenter les fichiers de traduction : | |
puis relancer ``powrap`` : |
Merci @dmerejkowsky ! |
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")