Skip to content
This repository has been archived by the owner on Apr 16, 2018. It is now read-only.

Update start and readme #11

Closed
wants to merge 2 commits into from
Closed

Update start and readme #11

wants to merge 2 commits into from

Conversation

fpagnoux
Copy link
Contributor

@fpagnoux fpagnoux commented Jul 1, 2016

  • Use mes-aides.ini confing file when running start.sh
  • Update REAMDE.md

It seems that only mes-aides uses this repository, so let's be honest and not pay genericity cost for nothing. The goal of this repository is then just to help to deploy an openfisca instance compatible with mes-aides, and start.sh doesn't need a parameter.

@MattiSG
Copy link
Member

MattiSG commented Jul 4, 2016

Well, the whole point of this repo was to ease the deployment of a production OpenFisca instance. Has your work with OpenFisca packaging made it redundant (i.e. can one now just pip install openfica)?
If yes, then let's reassess the importance of this repo as a whole.
If no, then I'd rather keep it generic. The fact that we right now are the only users doesn't make the need go away. Plus, I'd argue the genericity costs have already been paid, and passing a param is not that expensive.

@MattiSG
Copy link
Member

MattiSG commented Dec 28, 2016

The adaptation to Mes Aides has been made in betagouv/aides-jeunes#384, without the need to make this repo less generic, thanks to proper packaging of OpenFisca.

@MattiSG MattiSG closed this Dec 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants