Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Revalorisation de l'ASS au 1er avril 2018 #977

Merged
merged 4 commits into from
May 25, 2018
Merged

Revalorisation de l'ASS au 1er avril 2018 #977

merged 4 commits into from
May 25, 2018

Conversation

frtomas
Copy link
Contributor

@frtomas frtomas commented Apr 27, 2018

  • Évolution du système socio-fiscal.
  • Périodes concernées : à partir du 01/04/2018.
  • Zones impactées : prestations/minima_sociaux/ass
  • Détails :
    • Met à jour le montant journalier à taux plein de l'ASS en date du 01/04/2018

Ces changements :

  • Corrigent ou améliorent un calcul déjà existant.

@frtomas frtomas requested a review from guillett April 27, 2018 09:42
guillett
guillett approved these changes May 3, 2018
View reviewed changes
Copy link
Member

@guillett guillett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci @frtomas pour cette contribution.

Comme pour la pull request précédente mais remplacé ton commit de merge par un rebase.

J'ai aussi apporté quelques modifications que tu peux voir dans deux commits (cc5eefa et 704cb90).

@guillett
Copy link
Member

guillett commented May 3, 2018

À bien y réfléchir, il faudrait une mention à la circulaire qui donne le bon coefficient de revalorisation. J'ai envoyé un message à l'équipe responsable de http://circulaire.legifrance.gouv.fr pour savoir si il y a un lien stable.

@frtomas
Copy link
Contributor Author

frtomas commented May 3, 2018

Merci @guillett , j'attends ton retour concernant le lien

@guillett
Copy link
Member

@frtomas je n'ai pas eu de nouvelle, mais mergeons. Peux-tu faire un rebase et je merge ensuite. Merci.

@frtomas frtomas force-pushed the msa_reval_ass branch 2 times, most recently from 8256ced to 016bdfd Compare May 23, 2018 10:09
@frtomas
Copy link
Contributor Author

frtomas commented May 23, 2018

Bonjour @guillett , je viens de faire le rebase.

guillett
guillett reviewed May 23, 2018
View reviewed changes
tests/formulas/ass_04_2018.yaml Outdated
aah: 0
output_variables:
ass: 494.40
# ass
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commentaire sans valeur ajoutée

guillett
guillett reviewed May 23, 2018
View reviewed changes
Copy link
Member

@guillett guillett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci pour le rebase, j'ai fait quelques remarques supplémentaires. Les commentaires dans les PRs précédentes étaient super intéressants car ils ajoutaient de l'information sur l'ajustement qui était réalisé. Ici pas d'ajustement les commentaires n'ajoute pas de valeur.

Aussi, je vois que tu as fait des commits ayant le même nom. As-tu déjà fait un rebase interactif ? Cela permet de nettoyer un historique de commits avant de le merger à master ? Es-tu disponible 10/15min par téléphone pour le faire ensemble en pas à pas ?

tests/formulas/ass_04_2018.yaml Outdated
aah: 0
output_variables:
ass: 494.40
# ass
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commentaire sans valeur ajoutée

tests/formulas/ass_04_2018.yaml Outdated
- id: "parent2"
output_variables:
ass: 494.40
# ass
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commentaire sans valeur ajoutée

tests/formulas/ass_04_2018.yaml Outdated
aah: 0
output_variables:
ass: 253.60
# ass
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commentaire sans valeur ajoutée

tests/formulas/ass_04_2018.yaml Outdated
- id: "parent2"
output_variables:
ass: 312.80
# ass
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commentaire sans valeur ajoutée

tests/formulas/ass_04_2018.yaml Outdated
aah: 0
output_variables:
ass: 0
# ass
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commentaire sans valeur ajoutée

tests/formulas/ass_04_2018.yaml Outdated
- id: "parent2"
output_variables:
ass: 0
# ass
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commentaire sans valeur ajoutée

@frtomas
Copy link
Contributor Author

frtomas commented May 23, 2018

@guillett je n'ai jamais fait de rebase interactif, mais je ne pense pas avoir le temps de regarder ça avec toi aujourd'hui.
En revanche demain ça ne devrait pas poser de problèmes.

@frtomas frtomas added the contrib:msa Identification des sujets MSA label May 23, 2018
@guillett
Copy link
Member

Merci. Très bien, voyons ça demain.

guillett
guillett reviewed May 25, 2018
View reviewed changes
CHANGELOG.md Outdated
## 21.10.0 [#980](https://github.com/openfisca/openfisca-france/pull/980)

* Amélioration technique
* Détails :
- Adapte à Python 3

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Des caractères non visibles ont été ajoutés par erreur lors du rebase.

@guillett guillett merged commit 64c0881 into master May 25, 2018
@guillett guillett deleted the msa_reval_ass branch May 25, 2018 15:08
frtomas added a commit that referenced this pull request Jun 7, 2018
@frtomas
Prise en compte de la revalorisation ASS (#977) dans les résultats de…
ce3049f 
… tests
@bonjourmauko bonjourmauko added contrib:msa Identification des sujets MSA and removed contrib:msa Identification des sujets MSA labels Jul 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guillett guillett guillett approved these changes

Assignees
No one assigned
Labels
contrib:msa Identification des sujets MSA kind:evolution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@frtomas @guillett @bonjourmauko