-
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
Fix fonction publique #1219
Fix fonction publique #1219
Conversation
@Morendil : je t'ai demandé une revue en supposant que tu es le reviewer par défaut côté DINSIC/MesAides pour France mais si je me trompes, il faut me le dire ;-) |
@benjello C'est OK pour moi ! Je te conseille (pour les PR France en général) de ne pas t'embêter avec le bump Changelog/setup, pour moi c'est plus simple de le faire au moment de la review et juste avant de merger - le temps d'une review il y a toujours eu autre chose mergé entre temps. |
@Morendil : pourrais-tu approuver alors ? ;-) |
@benjello Je voulais dire, c'est OK pour moi de faire la review, et ça sera sans doute demain au mieux, début de semaine prochaine au pire. (edit semaine prochaine…) |
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.
Je ne vois ni test ni référence législative, pour pouvoir approuver ce serait utile de savoir quelles observations te font penser que le calcul est erroné dans ce domaine et comment tu as su dans quel sens il fallait le corriger.
Par ailleurs les corrections sur les contributions ne sont pas décrites dans le Changelog, ont-elles un rapport avec les modifications principales, sont-elles purement opportunistes ?
|
@benjello Je pensais à quelque chose comme "Décret n° 2010-761 du 7 juillet 2010" qui permet de se rendre compte quand on a pas d'expertise dans le domaine que cette notion d'indice majoré s'applique bien à ces populations… maintenant que j'ai appris ça, OK pour moi. |
b8e12bd
to
79de7c7
Compare
Merci de contribuer à OpenFisca ! Effacez cette ligne ainsi que, pour chaque ligne ci-dessous, les cas ne correspondant pas à votre contribution :)
prelevements_obligatoires/prelevements_sociaux/contributions_sociales/remplacement
prelevements_obligatoires/prelevements_sociaux/cotisations_sociales/travail_totaux
revenus/activite/salarie
Ces changements corrigent ou améliorent un calcul déjà existant.
Quelques conseils à prendre en compte :
[ ] Documentez votre contribution avec des références législatives.[ ] Mettez à jour ou ajoutez des tests correspondant à votre contribution.setup.py
.CHANGELOG.md
.Et surtout, n'hésitez pas à demander de l'aide ! :)