Byron / bcore

powerful application framework built to integrate, not separate
http://byron.github.io/bcore/
Other
8 stars 2 forks source link

review legacy_inherit_env #7

Closed Byron closed 10 years ago

Byron commented 10 years ago

This one need proper review, and either it is removed from the feature set, or it will trigger a big warning when used.

It shouldn't be needed, and people can actually fall for it not knowing it is in fact enabled.

Byron commented 10 years ago

It is called 'inherit_environment' and is documented accordingly. It's off by default, so people turning it on probably have a reason to do so. As it's only working for the root package, there is literally no chance it's pulled in accidentally.