Enforce pep8 in server-backend

ClubCedille / jardiniot

Système pour des buckets autonomes servant à faire pousser des plantes

GNU General Public License v3.0

13 stars 11 forks source link

Enforce pep8 in server-backend #94

Closed notarock closed 4 years ago

notarock commented 4 years ago

Applique PEP8 sur tout le code python en utilisant autopep8
Ajoute un script bien simple qui lint tous les fichiers
Le CI plante si autopep --diff affiche du texte. Cela veut dire que le code n'est pas conforme à PEP8.

notarock commented 4 years ago

Ferme 1/3 de #89

notarock commented 4 years ago

faut trouver une solution pour les doc strings.

Probablement suivre pep8 et mettre la docstring dans la fonction, m'a le fix quand je vais avoir le temps

AXDOOMER commented 4 years ago

Le lint a pas poigné l'erreur des ", est-ce que c'est possible de configurer le lint pour qu'il attrape ces erreurs?

notarock commented 4 years ago

Le lint a pas poigné l'erreur des ", est-ce que c'est possible de configurer le lint pour qu'il attrape ces erreurs?

Je vais checker, mais je pense que le linter crois que le " fait parti de la docstring.

AXDOOMER commented 4 years ago

Pourquoi est-ce que j'aime pas PEP8 déjà? Ah oui, les retours de ligne forcés à 79 caractères. Encore un choix qui n'est pas basé sur la science et l'aspect pratique. Le linter enforce toutes ces règles sans esprit critique.

Screenshot_2020-05-22_10-20-43

AXDOOMER commented 4 years ago

L'origine de "80 colonnes par ligne de code" :

notarock commented 4 years ago

J'ai fix le 120 char, la docstring, toute la patente

AXDOOMER commented 4 years ago

Nice, je vais review prochainement.

berbiche commented 4 years ago

@notarock ping

mikefaille commented 4 years ago

beau