openfisca / openfisca-france-fiscalite-miniere

French mining tax system for OpenFisca
GNU Affero General Public License v3.0
3 stars 2 forks source link

Ajoute les RDCM à toutes les substances et met à jour surface_totale #27

Closed sandcha closed 2 years ago

sandcha commented 2 years ago

Ces changements :

Quelques conseils à prendre en compte :

sandcha commented 2 years ago

Correction des tests (ignore flake8 sur points) dans un commit à venir.

sandcha commented 2 years ago

Rebase en cours.

sandcha commented 2 years ago

Merci pour vos retours !

J'avoue que la quantité de code rend la lecture de la PR difficile, je l'approuve en l'état, ça nous permettra de l'intégrer et de voir si il y'a des bugs plus tard.

J'ai fais une proposition de mutualisation du code ici #28 si jamais on veut travailler à la lisibilité du code

Effectivement, faire toutes les substances d'un coup n'était probablement pas le plus praticable. Pardon pour ça. Ok pour merge et amélioration avec la #28. Sur la duplication de code, l'idée est que :