Tarrasch / zsh-autoenv

Autoenv for zsh

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Use AUTOENV_ prefix for variables

blueyed opened this issue · comments

What's the name of the project?

While going wild on the source, I've used the "dotenv" name for vars etc, because it was used internally already.

Should this get refactored to use AUTOENV_* vars?
This would be nice to have it as a drop-in replacement for https://github.com/kennethreitz/autoenv - as long as it uses the same mechanism for AUTOENV_ENV_FILENAME etc.

I think that's a great idea. I took the "dotenv" namespace from Joshua's implementation, but I much rather conform to whatever kennethreitz/autoenv uses instead. Would you like to send a PR for this? :)

Would you like to send a PR for this? :)

Sure, but I would build it on top of the "improvements" and/or "varstash" branch/PR.
It would be great to get feedback on those from you.

Ah, I'm reviewing it now. :)

I'll look into this when the varstash branch has been merged, in a few days probably.

Makes sense. :)

Merged into master.