Skip to content

Update flake8 parameters

Ronan Lefol requested to merge rolef53-develop-patch-31962 into develop

Il y a des problèmes de compatibilité entre flake8 et Black

  • Line length: Black vise ~88, Flake8 vise < 79
  • Black traite : comme un binary operator (espace avant et après), flake8 ne veux pas d’espace avant.

Pour le deuxième point, flake8 ne suit tout simplement pas les conventions PEP8 donc l'utilisation de --ignore=E203 est valide selon moi

Le premier point est assez basé sur opinion, mais le consensus en-ligne semble être de laisser black faire ses choses et déborder un peu au besoin. Donc la règle --max-line-length=88. Cette règle pour aussi être poussée un peu plus loin au besoin.

À voire si tout le monde est d'accord.

Edited by Ronan Lefol

Merge request reports

Loading