Ajout de l'usage 'powrap' pour l'indentation des fichiers by Fipaddict · Pull Request #1764 · python/python-docs-fr · GitHub
Skip to content

Ajout de l'usage 'powrap' pour l'indentation des fichiers#1764

Closed
Fipaddict wants to merge 1 commit into
python:3.10from
Fipaddict:contributing
Closed

Ajout de l'usage 'powrap' pour l'indentation des fichiers#1764
Fipaddict wants to merge 1 commit into
python:3.10from
Fipaddict:contributing

Conversation

@Fipaddict

Copy link
Copy Markdown
Contributor

Suite à la discussion sur https://discuss.afpy.org/t/resolu-erreur-1-lors-du-wrap/587 , je propose d'ajouter l'usage de la commande powrap -m pour réindenter les fichier (vs make wrap qui ne fait que vérifier)

@jeanas

jeanas commented Nov 3, 2021

Copy link
Copy Markdown
Collaborator

@JulienPalard

Copy link
Copy Markdown
Member

+1 pour changer make wrap et lui faire rewrapper, ça veut dire qu'il faut changer make verifs pour lui faire juste executer powrap avec le --check quand même, on est 3 a trouver contre-intuitif le fait que make wrap utilise le --check.

@Fipaddict

Copy link
Copy Markdown
Contributor Author

Votre proposition me semble aussi plus intéressante que ma PR (qui sera alors complètement hors sujet).
Du coup, je la ferme ? Quelle est la bonne pratique sur le sujet ?

@jeanas

jeanas commented Nov 4, 2021

Copy link
Copy Markdown
Collaborator

C'est comme vous voulez : soit vous vous sentez de changer cette PR pour faire ce qui a été résolu, et vous rajoutez simplement un autre commit dans la branche, soit vous préférez le laisser à quelqu'un d'autre (je peux m'en occuper sans problème) et dans ce cas vous pouvez fermer la PR.

@Fipaddict

Copy link
Copy Markdown
Contributor Author

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.

3 participants