Skip to content

Ajout de pre-commit#2697

Merged
benoit-cty merged 7 commits intomasterfrom
feat/pre-commit
Feb 12, 2026
Merged

Ajout de pre-commit#2697
benoit-cty merged 7 commits intomasterfrom
feat/pre-commit

Conversation

@benoit-cty
Copy link
Contributor

@benoit-cty benoit-cty commented Feb 10, 2026

  • Amélioration technique.
  • Périodes concernées : toutes.
  • Zones impactées : *.
  • Détails :
    • Mise en place de pre-commit pour contrôler la mise en forme lors du commit.
    • Documenter comment l'installer.
    • Correction des erreurs de lint existantes.
    • Exclusion des erreurs BLK100 Black would make changes..

Dans l'idéal je pense qu'il faudrait passer à ruff mais ici le but n'est pas de changer le format de ce qui existe, juste avoir un make check-style qui ne retourne plus aucune erreur.

Et avoir un pre-commit pour permettre aux développeurs de corriger les erreurs avant le commit.


Ces changements modifient des éléments non fonctionnels de ce dépôt.


Quelques conseils à prendre en compte :

@benoit-cty
Copy link
Contributor Author

benoit-cty commented Feb 10, 2026

Pour la revue, il faut regarder en particulier les fichiers à la racine :

  • pyproject.toml
  • CONTRIBUTING.md
  • .pre-commit-config.yaml

Pour le reste ce n'est que l'application du format souhaité par OpenFisca-France mais qui n'était pas respecté.

@benoit-cty benoit-cty changed the title Ajout de pre-commit Ajout de pre-commit et suppression de Conda Feb 10, 2026
@benoit-cty benoit-cty changed the title Ajout de pre-commit et suppression de Conda Ajout de pre-commit Feb 10, 2026
@benoit-cty
Copy link
Contributor Author

La suppression de Conda a été déplacée dans #2699

Copy link
Contributor

@clallemand clallemand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai testé en local, tout semble bien fonctionner ! Merci

@benoit-cty benoit-cty enabled auto-merge February 12, 2026 09:52
@benoit-cty benoit-cty merged commit 48b42a1 into master Feb 12, 2026
36 checks passed
@benoit-cty benoit-cty deleted the feat/pre-commit branch February 12, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants