Skip to content

enh: code with doctrings and type hints

Comme ça touche un peu à tout, j'ai pas trop suivi les bonnes pratiques :/ et j'ai un peu inclus des trucs qui sont dans deux autres tickets : #154 et #176 (en plus du ticket initial #169).

Je documente où j'en suis dans chaque ticket par contre.

Cette merge request inclus donc :

  • L'utilisation de f-strings quand elles n'étaient pas utilisées (#176) ;
  • L'utilisation de la nouvelle sémantique de Flask-SQLAlchemy/SQLAlchemy (#154) quand ce n'était pas le cas ;
  • Ajout de docstrings sur les modules/fonctions/classes ;
  • Ajout d'annotations de type (variables/fonctions/classes) ;
  • Ajout de tests unitaires pour les éléments qui ont été touchés ;
  • Quelques modifications mineures (de bonnes pratiques) :
    • Rangement des imports ;
    • Renommage des fonctions qui avaient le même nom qu'une fonction de Python ;
    • etc.

⚠️ j'ai remplacé _ par trans pour pouvoir utiliser _ pour des variables dont on se fiche du nom, mais c'est en contradiction avec ce qui est dit dans la documentation de Traduction (qui préconise de ne pas utiliser lazy_gettext, car trop verbeux) que j'ai vu après coup.

Edited by ornanovitch

Merge request reports

Loading