chairemobilite / evolution

Online survey platform for travel survey
MIT License
4 stars 9 forks source link

Generator: Format files, remove error when no inputType and add better error message #702

Closed samuel-duhaime closed 1 month ago

samuel-duhaime commented 2 months ago

Pull request

Description

Format files, remove error when no inputType and add better error message.

Add scripts to use yarn format:python Format all the Python files with Black Add Black package in dev

Justification

It was hard to debug the Generator when the Excel file was modified with much error.

samuel-duhaime commented 1 month ago

C'est quoi le formatage des fichiers utilisé ici? Est-ce ton éditeur qui le fait automatiquement ? Ce qui veut dire que qqn d'autre pourrait avoir un formatage différent. Ou alors il y a une commande comme yarn format pour le faire automatiquement de la même manière pour tous?

@tahini Non, en effet, il n'y a pas de formatage automatique avec Python. C'était mon éditeur qui a fait cela.

Je crois qu'on peut ajouter cette commande au prettier pour y parvenir, mais je n'ai pas testé.

"[python]": {
    "editor.defaultFormatter": "ms-python.python"
  }

Je vais aller tester.

samuel-duhaime commented 1 month ago

@tahini

Prettier n'existe pas pour Python.

Okay, apparament ce code était pour le formatter de utopep8.

Il y a aussi black qui est populaire. Je peux essayer de tester et d'installer cela rapidement.

samuel-duhaime commented 1 month ago

@tahini J'ai installé une configuration pour formatter avec Black qui marche plutôt bien. Pas besoin d'avoir une configuration pour le configurer.

tahini commented 1 month ago

@samuel-duhaime l'idée c'est de ne pas changer le formatage au gré des gens/éditeurs et qu'on ait tous le même format. C'est la beauté de la commande yarn format. On oublie de la rouler 80% du temps, ce qui donne des commits intitulés "Run yarn format on the code", mais être plus disciplinés, on le saurait et on aurait tous la même chose!