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.

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

Rename aide_logement_participation_personelle to aide_logement_partic… #718

Merged
merged 2 commits into from
Apr 26, 2017

Conversation

SophieIPP
Copy link
Contributor

@SophieIPP SophieIPP commented Mar 23, 2017

…ipation_personnelle

Typo

  • Évolution du système socio-fiscal.
  • Périodes concernées : toutes.
  • Zones impactées : prestations/aides_logement.
  • Détails :
    • Renomme aide_logement_participation_personelle en aide_logement_participation_personnelle.
    • Le nom de la variable comportait une faute d'orthographe.

Ces changements :

  • Impactent l'API publique d'OpenFisca France (par exemple renommage ou suppression de variables).

EDIT: format

@SophieIPP
Copy link
Contributor Author

@MattiSG @fpagnoux Si la PR n'est pas dans les règles de l'art (j'ai essayé de suivre au mieux les guidelines mais certaines choses n'étaient pas très claires), merci de me dire comment l'améliorer :)

@fpagnoux
Copy link
Member

Bonjour Sophie,

Merci pour la correction ! Les tests ne passent pas car à chaque évolution il est nécessaire de documenter les évolutions et de faire évoluer le numéro de version.

Le format de documentation des évolutions est documenté ici. J'ai édité la description de la PR pour que ça colle, il ne reste plus qu'à le mettre dans le CHANGELOG.

N'hésite pas si tu as une question précise sur quelque chose qui n'est pas clair.

@benjello benjello removed the request for review from MattiSG April 18, 2017 10:29
@benjello
Copy link
Member

@fpagnoux : est-ce ok désormais ?

@fpagnoux fpagnoux force-pushed the fix-typo-participation-personnelle branch 2 times, most recently from fd325cb to fe774a7 Compare April 18, 2017 17:47
@benjello benjello requested a review from MattiSG April 19, 2017 05:42
CHANGELOG.md Outdated
@@ -1,6 +1,14 @@
# Changelog

### 16.1.1 - [#632](https://github.com/openfisca/openfisca-france/pull/632)
## 17.0.0 - [#718](https://github.com/openfisca/openfisca-france/pull/718))
Copy link
Member

Choose a reason for hiding this comment

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

Il y a une parenthèse fermante en trop 😉

SophieIPP and others added 2 commits April 25, 2017 12:35
…ipation_personnelle

Typo

# Conflicts:
#	openfisca_france/model/prestations/aides_logement.py
@benjello benjello force-pushed the fix-typo-participation-personnelle branch from 4d160f6 to a0798ef Compare April 25, 2017 10:40
@benjello
Copy link
Member

@fpagnoux / @MattiSG: j'ai essayé de rebaser proprement (et je crois l'avoir fait) mais je n'ai pu pusher qu'en forçant. Je vous laisser inspecter l'arbre avant de merger.

@benjello benjello merged commit 4340df4 into master Apr 26, 2017
@benjello benjello deleted the fix-typo-participation-personnelle branch April 26, 2017 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants