Correction de l'exonération de la CSG et CRDS pour les non-résidents

[Nodraak]

Fix #2104.

---

Merci de contribuer à OpenFisca ! Effacez cette ligne ainsi que, pour chaque ligne ci-dessous, les cas ne correspondant pas à votre contribution :)

• Évolution du système socio-fiscal. | Amélioration technique. | Correction d'un crash. | Changement mineur.
• Périodes concernées : toutes. | jusqu'au JJ/MM/AAAA. | à partir du JJ/MM/AAAA.
• Zones impactées : chemin/vers/le/fichier/contenant/les/variables/impactées.
• Détails :
  • Description de la fonctionnalité ajoutée ou du nouveau comportement adopté.
  • Cas dans lesquels une erreur était constatée.

Cf CHANGELOG.md.

---

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

• Modifient l'API publique d'OpenFisca France (par exemple renommage ou suppression de variables).
• Ajoutent une fonctionnalité (par exemple ajout d'une variable).
• Corrigent ou améliorent un calcul déjà existant.
• Modifient des éléments non fonctionnels de ce dépôt (par exemple modification du README).

---

Quelques conseils à prendre en compte :

• Jetez un coup d'œil au guide de contribution.
• Regardez s'il n'y a pas une proposition introduisant ces mêmes changements.
• Documentez votre contribution avec des références législatives.
• Mettez à jour ou ajoutez des tests correspondant à votre contribution.
• Augmentez le numéro de version dans setup.py.
• Mettez à jour le CHANGELOG.md.
• Assurez-vous de bien décrire votre contribution, comme indiqué ci-dessus

Et surtout, n'hésitez pas à demander de l'aide ! :)

[Nodraak]

Hello @benjello

Ma PR est prête. Comment faire pour lancer la CI ?

[benjello]

Ma PR est prête. Comment faire pour lancer la CI ?

Elle se lance toute seule normalement. cc @sandcha @MattiSG ?

[MattiSG]

Comme elle provient d'un contributeur externe à l'organisation OpenFisca, cette PR devrait nécessiter une autorisation, mais l'interface pour autoriser n'apparaît pas 😯
J'ai revu les réglages à la fois pour le dépôt et pour l'organisation, sans succès. Je ne sais pas comment débloquer la situation pour le futur.
J'ai poussé manuellement la branche du fork dans la branche nodraak-fix-2104-non-residents dans l'organisation, ce qui a déclenché la CI. Cela devra a priori être fait systématiquement pour la suite, ce qui n'est pas très confortable.

Bienvenue @Nodraak et merci pour ta contribution ! 🙂
Si tu prévois d'être un contributeur récurrent affilié à un partenaire (IPP par exemple), le plus simple serait que tu rejoignes l'organisation OpenFisca.

[Nodraak]

Comme elle provient d'un contributeur externe à l'organisation OpenFisca, cette PR devrait nécessiter une autorisation, mais l'interface pour autoriser n'apparaît pas hushed J'ai revu les réglages à la fois pour le dépôt et pour l'organisation, sans succès. Je ne sais pas comment débloquer la situation pour le futur. J'ai poussé manuellement la branche du fork dans la branche nodraak-fix-2104-non-residents dans l'organisation, ce qui a déclenché la CI. Cela devra a priori être fait systématiquement pour la suite, ce qui n'est pas très confortable.

Bienvenue @Nodraak et merci pour ta contribution ! slightly_smiling_face Si tu prévois d'être un contributeur récurrent affilié à un partenaire (IPP par exemple), le plus simple serait que tu rejoignes l'organisation OpenFisca.

Ah d'accord, je comprend. C'est embêtant ... Pourquoi cette limitation ? La CI coûte cher ?

Je ne suis pas affilié à quoi que ce soit, et je ne peux pas prédire la récurence ou non de mes contributions : je suis un simple curieux qui a vu de la lumière et qui est entré ! Mais si vous m'invitez dans l'organisation OpenFisca, j'accepterai avec joie ;)

[benjello]

Utiliser le nom de la variable openfisca, pour la variable python pour éviter les confusions.

[Nodraak]

Tu veux dire, faire resident_eee_hors_france = ... ? Ca me semble être une très mauvaise idée, étant donné qu'il y a une condition avec la date ainsi qu'un not_(), mais je ne comprend pas ce que tu pourrais vouloir dire d'autre ...

[MattiSG]

Pourquoi cette limitation ?

Simplement parce que le code exécuté en CI peut être modifié par une PR pour extraire des secrets ou effectuer du travail type minage de cryptos, d'où le modèle de sécurité qui nécessite une approbation avant exécution 🙂

Je ne suis pas affilié à quoi que ce soit, et je ne peux pas prédire la récurence ou non de mes contributions

Alors un grand merci @Nodraak pour la contribution (et @benjello pour la revue 👍) !
À ce stade, vu le coût administratif, nous limitons l'ajout dans l'organisation aux contributeurs réguliers 🙂

Je vais pousser à nouveau la branche pour déclencher la CI.

Pour les itérations suivantes, et afin de permettre le suivi par les reviewers de l'évolution du code, merci de ne pas faire de force push mais uniquement des commits supplémentaires 🙂

[benjello]

Il vaut mieux éviter de manipuler des GroupPopulation et passer par des variables. Elles permettent la gestion des changements de législation dans le temps de façon plus naturelle.