os.environb suppport?

[abathur]

os.environb suppport?

[marcinzaremba]

That's a great idea. We can support both actually. If source is provided on class then it overrides the settings on Variable.
However in this case (when source would be os.environb) my thinking is it is undesirable to have it globally because it would be very painful for user to convert from bytes to str for example:

# Given that the transform func on Variable is the default cast
>>> from ecological import config
>>> config.cast(b'My value', str)
"b'My value'"

So suddenly all strings become screwed when you switch globally to os.environb and then you would new to explictly use Variable on each of them to do some other transform.

[marcinzaremba]

For me it sounds like really easy change that can be implemented in a very generic, useful way.

First introduce a concept of source on Variable class and pass it as the constructor parameter; the protocol here would be source is a dictionary that Variable reads values from where os.environ stays the default one. Then utilizing os.environb would be trivial from user POV:

class Configuration(ecological.AutoConfig):
    my_var1: str = ecological.Variable("my_var1", source=os.environb)

[thilp]

Would there be a way to also specify source as "class argument", just like prefix? (I think you would have all your values coming from the same dict much more often than from different dicts.)