Project management #49820
adoption de black
Status:
Fermé
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
04 January 2021
Due date:
% Done:
0%
Estimated time:
Patch proposed:
No
Planning:
No
Club:
No
Description
Comme ça a été fait dans chrono, adopter black dans les différents modules.
Procédure, à discuter/valider,
- par module
- assurer qu'un maximum des patchs proposés soient validés/intégrés
- commit qui fait black (exemple : "trivial: apply black")
- commit qui ajoute un fichier .pre-commit-config.yaml et l'info associée au fichier README (si présent) + qui crée un fichier .git-blame-ignore-revs avec la révision du commit précédent (ex: "misc: add black files/notes")
Texte actuellement présent dans le README de chrono :
Code Style ---------- black is used to format the code, using thoses parameters: black --target-version py37 --skip-string-normalization --line-length 110 There is .pre-commit-config.yaml to use pre-commit to automatically run black before commits. (execute `pre-commit install` to install the git hook.)
Questions
- on garde le --target-version py35 ou on le passe à 3.7 vu qu'on ne devrait plus se soucier de stretch ?
- → on met py37
- on pose une échéance ?
History
Updated by Frédéric Péters over 3 years ago
- Description updated (diff)
Je viens de mettre à jour la description pour réunir les modifications dans deux commits et noter la décision sur l'utilisation de --target-version py37.
Updated by Frédéric Péters over 3 years ago
Sur ce modèle, j'ai fait passerelle-grandlyon-cartads-cs, passerelle-montpellier-* et publik-base-theme.
Updated by Frédéric Péters over 3 years ago
Il y en a d'autres depuis mais là je note majeur que je viens de faire ça sur w.c.s.
Updated by Frédéric Péters over 3 years ago
Je viens de le faire sur combo (ce que je pensais avoir déjà fait), et j'ai ajouté un peu partout dans tox.ini l'exécution de black, comme validé dans #50927.
Updated by Frédéric Péters over 2 years ago
- Status changed from Nouveau to Fermé
C'est en place partout depuis un certain temps.