Project

General

Profile

Project management #49820

adoption de black

Added by Frédéric Péters over 3 years ago. Updated over 2 years ago.

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

#1

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.

#2

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.

#3

Updated by Frédéric Péters over 3 years ago

et eobuilder et fargo et welco.

#4

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.

#5

Updated by Frédéric Péters over 3 years ago

  • Description updated (diff)
#6

Updated by Lauréline Guérin over 3 years ago

\o/

#7

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.

#8

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.

Also available in: Atom PDF