Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
JulienPalard merged 3 commits intopython:3.9fromdmerejkowsky:suggestions
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 ungit add <fichier.po> avant legit commit

@pythonpython deleted a comment fromPyDocTeurMay 20, 2021
@dmerejkowsky
Copy link
ContributorAuthor

J'ai rajouté legit 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 changesMay 24, 2021
@PyDocTeur
Copy link

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


Disclaimer

Je suis un robot fait par l'équipe del'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 theTranslation 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

@JulienPalardJulienPalard 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 !

@@ -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 parGitHUb
le résultat n'est pas toujours convenable pour ``powrap``. Si cela arrive, vous pouvez récupérer le commit fait parGitHub

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

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
puisré-indenter les fichiers de traduction :
puisrelancer ``powrap`` :

@JulienPalardJulienPalard merged commit60ed982 intopython:3.9May 24, 2021
@JulienPalard
Copy link
Member

Merci@dmerejkowsky !

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@JulienPalardJulienPalardJulienPalard approved these changes

@christopheNanchristopheNanchristopheNan left review comments

Assignees
No one assigned
Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@dmerejkowsky@christopheNan@PyDocTeur@JulienPalard

[8]ページ先頭

©2009-2025 Movatter.jp