search

Tarrasch / zsh-autoenv

Autoenv for zsh
700 stars 30 forks source link

Rename _autoenv_event, _autoenv_from_dir, _autoenv_to_dir? #11

Closed blueyed closed 9 years ago

blueyed commented 9 years ago

Should the vars meant to be used by the .env file get renamed, e.g. without leading _ and/or uppercased?

See https://github.com/Tarrasch/zsh-autoenv/blob/master/autoenv.zsh#L198

Tarrasch commented 9 years ago

Yea, I guess the _autoenv suffix should be only for internal stuff. If a variable is meant to be set by the user, I suppose it should be just be suffixed with autoenv. I'm not sure about upper case, do they do upper case anywhere else?

blueyed commented 9 years ago

Upper case is used for the config vars. I will remove the underscore prefix then.

Tarrasch commented 9 years ago

Well, I think we should choose whatever is the most common convention.

Tarrasch commented 9 years ago

Awesome initiative! :)