Projet

Général

Profil

Project management #49820

adoption de black

Ajouté par Frédéric Péters il y a plus de 3 ans. Mis à jour il y a plus de 2 ans.

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

#1

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.

#2

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.

#3

Mis à jour par Frédéric Péters il y a plus de 3 ans

et eobuilder et fargo et welco.

#4

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.

#5

Mis à jour par Frédéric Péters il y a environ 3 ans

  • Description mis à jour (diff)
#6

Mis à jour par Lauréline Guérin il y a environ 3 ans

\o/

#7

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.

#8

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.

Formats disponibles : Atom PDF