-
Notifications
You must be signed in to change notification settings - Fork 101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mets à jour le crédit d'impôt pour la transition energétique (IR 2019 sur revenus 2018) #1329
Conversation
aefa399
to
817ac7e
Compare
|
||
maries_ou_pacses = foyer_fiscal('maries_ou_pacses', period) | ||
personnes_a_charge = foyer_fiscal('nb_pac2', period) | ||
rfr = foyer_fiscal('rfr', period) # noqa F841 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
La variable rfr
n'est pas utilisée (c'est ce qu'indique le commentaire noqa
qui vient inhiber le linter), supprimer cette ligne dans la version 2017 et dans celle-ci.
rfr = foyer_fiscal('rfr', period) # noqa F841 | ||
P = parameters(period).impot_revenu.credits_impot.quaenv | ||
|
||
depenses_transition_energetique = ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dans la formule 2018 comme dans la formule 2017 et les années antérieures, on gagnerait énormément à écrire quelque chose comme
cases_depenses = ['f7aa', 'f7ad', …, 'f7cb']
depenses_transition_energetique = sum([foyer_fiscal(case, period) for case in cases_depenses])
et supprimer toutes les lignes qui ne servent qu'à introduire les variables f7aa, etc.
@Morendil Est-ce que les changements que j'ai fait correspondent à ce que tu avais en tête ? |
@claireleroy Oui c'est bon pour moi, tu prends rebase et merge ou je prends? Si tu prends, attention aux trucs un peu chiants du changelog (le numéro de PR, le nombre de # correspondant à la montée de versions, guide de migration si breaking…). |
@Morendil Ok je prends, je vais essayer de faire attention :) |
434e0a0
to
82be9ba
Compare
@Morendil Ca te sembles ok ? |
model/prelevements_obligatoires/impot_revenu/
.Ces changements :