edward icon indicating copy to clipboard operation
edward copied to clipboard

Support dotenv files

Open fromz opened this issue 7 years ago • 5 comments

Has anyone put any thought into supporting dotenv files for environment variable management natively? I'm happy to do the dev work if we agree it's a good idea

fromz avatar Feb 21 '18 21:02 fromz

Interesting idea! Do you have any thoughts on particular use cases for this feature? I think it could be useful, but there's potential for clashes or confusing behavior when launching services that already use dotenv internally.

theothertomelliott avatar Feb 22 '18 01:02 theothertomelliott

Yeah, so we as a dev team heavily rely on edward for our services. Some of us have different environment variable configurations, and it would be good to be able to easily maintain that between machines without having to manage dotenv manually. I think if we made the configuration very distinct between dotenv and the existing "env" configuration, it should be fairly clear, especially if we spell out the order of precedence - e.g. shell set environment variables first, then edward "env" second, then dotenv last, or some such.

fromz avatar Feb 22 '18 01:02 fromz

Sounds like a decent order of precedence. Would you imagine dotenv being an "always on" feature, or something you'd enable for a specific Edward config, group or service?

Are the different variable configurations on your team personal settings per developer (perhaps being manually set and listed in gitignore)?

theothertomelliott avatar Feb 22 '18 01:02 theothertomelliott

I'd say per service initially and then maybe add it per group. What do you think? Happy to send a pr in the next few days if you agree

On Thu., 22 Feb. 2018, 11:51 am Tom Elliott, [email protected] wrote:

Sounds like a decent order of precedence. Would you imagine dotenv being an "always on" feature, or something you'd enable for a specific Edward config, group or service?

Are the different variable configurations on your team personal settings per developer (perhaps being manually set and listed in gitignore)?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/yext/edward/issues/151#issuecomment-367540548, or mute the thread https://github.com/notifications/unsubscribe-auth/AANiUNgI8b87DfQoSOFK-5qOHmpxp387ks5tXMgFgaJpZM4SOY7g .

fromz avatar Feb 22 '18 01:02 fromz

Sounds good, look forward to seeing your PR! We've been adding features on the develop branch before merging for versioned releases.

theothertomelliott avatar Feb 22 '18 02:02 theothertomelliott