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

Added "automapping" feature for environment variables #60

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

phansys
Copy link

@phansys phansys commented Jul 16, 2014

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes

For more details, see README.md.

@stof
Copy link
Member

stof commented Jul 16, 2014

I will need a bit of time to review this in detail (I don't have this time right now), but the feature looks interesting

@phansys
Copy link
Author

phansys commented Jul 16, 2014

Thank you @stof.
Take your time.

@phansys phansys force-pushed the master branch 3 times, most recently from e828133 to 42ba5d8 Compare September 17, 2014 02:15
@kcivey
Copy link

kcivey commented Oct 23, 2014

It seems like this would be better done with a prefix (or at least an option for having one) the way Symfony does it (along with translating double underscore to dot):
http://symfony.com/doc/current/cookbook/configuration/external_parameters.html

The prefix could be defined in the incenteev-parameters.

@phansys
Copy link
Author

phansys commented Oct 23, 2014

I agree with your comment @kcivey. If @stof also agree, I could apply this change. This would be something like env-prefix or similar.

@phansys phansys closed this Oct 23, 2014
@phansys phansys reopened this Oct 23, 2014
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.

None yet

3 participants