Project management #49820
adoption de black
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
04 janvier 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Non
Planning:
Non
Club:
Non
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 ?
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Description mis à jour (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.
Mis à jour par Frédéric Péters il y a plus de 3 ans
Sur ce modèle, j'ai fait passerelle-grandlyon-cartads-cs, passerelle-montpellier-* et publik-base-theme.
Mis à jour par Frédéric Péters il y a environ 3 ans
Il y en a d'autres depuis mais là je note majeur que je viens de faire ça sur w.c.s.
Mis à jour par Frédéric Péters il y a environ 3 ans
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.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Nouveau à Fermé
C'est en place partout depuis un certain temps.