-
Notifications
You must be signed in to change notification settings - Fork 97
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
Amélioration de la lisibilité de variables de la fiscalité #1027
Conversation
@ClaireLeroyIPP : ton avis peut aider. Rajouter des renommages et/ou donner ton avis sur ceux-ci. |
@ClaireLeroyIPP as-tu des suggestions d'ajouts ? :) Merci ! |
@guillett Quelques idées :
Il y a un arbitrage nom de variable court versus nom de variable clair, je ne sais pas trop quel est la meilleure solution.. C'est pas sur que ce soit le nom le plus long. C'est tout ce que je vois à renommer là tout de suite, et je suis d'accord avec les deux que tu proposent ! :) |
Merci beaucoup @ClaireLeroyIPP. Sans liste explicite de renommage, je préfère merger les PRs et laisser à plus tard une seconde phase de renommage. @benjello, @ClaireLeroyIPP WDYT ? |
No objection. Mais rassemblons ces renommage au bon endroit. |
@benjello, @ClaireLeroyIPP l'un d'autre vous peut-il faire une review ? Merci |
@guillett Ca me parait prêt à merger ! |
Depends on #1025
prelevements_obligatoires/impot_revenu/ir
.rev_cap_lib
enrevenus_capitaux_prelevement_liberatoire
rev_cap_bar
enrevenus_capitaux_prelevement_bareme
Ces changements (effacez les lignes ne correspondant pas à votre cas) :