Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Test and fix for issue #414 #415

Merged
merged 2 commits into from Mar 1, 2012

Conversation

Projects
None yet
2 participants
Contributor

dave-shawley commented Mar 1, 2012

Added a test to regress #414 and a simple fix.

dave-shawley added some commits Mar 1, 2012

Fixed silent keyword arg to config.from_envvar.
The ``silent`` keyword argument to Config.from_envvar was not being honored
if the environment variable existed but the file that it mentioned did not.
The fix was simple - pass the keyword argument on to the underlying call to
``from_pyfile``.  I also noticed that the return value from ``from_pyfile``
was not being passed back so I fixed that as well.

@rduplain rduplain merged commit 8d7ca29 into pallets:master Mar 1, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment