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

Retire le fichier model/datatrees #757

Merged
merged 3 commits into from
May 22, 2017
Merged

Retire le fichier model/datatrees #757

merged 3 commits into from
May 22, 2017

Conversation

Anna-Livia
Copy link
Contributor

@Anna-Livia Anna-Livia commented May 22, 2017

Connected to #738

  • Changement mineur.
  • Périodes concernées : toutes.
  • Zones impactées :
    -- In : openfisca-france/openfisca_france/france_taxbenefitsystem.py
    ---- Deleted line 10 : rom .model import datatrees
    ---- Deleted line 27 : columns_name_tree_by_entity = datatrees.columns_name_tree_by_entity
    -- Deleted openfisca_france/scripts/generate_columns_tree.py
    -- Deleted openfisca_france/model/datatrees.py
  • Détails :
    Retirer le fichier model/datatrees Retirer le fichier model/datatrees #738

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

  • Modifient des éléments non fonctionnels de ce dépôt (par exemple modification du README).

@Anna-Livia Anna-Livia requested a review from fpagnoux May 22, 2017 11:29
@@ -7,7 +7,7 @@

from .entities import entities
from . import decompositions, scenarios
from .model import datatrees

Copy link
Member

@fpagnoux fpagnoux May 22, 2017

Choose a reason for hiding this comment

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

This line break is not necessary

@Anna-Livia Anna-Livia merged commit 5f06cb2 into master May 22, 2017
@Anna-Livia Anna-Livia deleted the Retirer_datatrees branch May 22, 2017 15:48
@MattiSG MattiSG changed the title Retirer le fichier model/datatrees Retire le fichier model/datatrees May 26, 2017
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.

2 participants