Skip to content

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

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