Lucky aware loading

[hovsater]

This is an attempt at implementing Lucky aware loading. I decided to use overloading instead of introducing LuckyEnv.load!. I extended the macro to now take a env_file argument that associates a given path with the environment.

Not sure if this is the approach we want to pursue, but it's a starting point. 🙂

[hovsater]

@jwoertink is this inline with what you had in mind? 🙂

[jwoertink]

I'll have to come back to this. I looked over it briefly, and I think it looks ok, but I forget what I was originally intending... 😂 Thanks for giving this a shot!

[hovsater]

Sounds good! If we deem it unnecessary, I'm happy just closing it. No hard feelings, haha. 😅

[wout]

@hovsater This is great. I'd love to see this merged but I think one improvement would make it perfect. Here's what I would like to suggest.

It .env could be considered the "global" file, which is loaded in all environments, and a .env.development would be added, then we could also avoid duplication of env vars.

A bit like it's outlined here: https://github.com/bkeepers/dotenv#what-other-env-files-can-i-use. Although, that approach is going too far in my opinion.

[jwoertink]

Sorry for the late review. This just came up again in the Lucky Discord. I love @wout's idea, but I'm also ok with maybe merging this as is, and we can update it with that idea after?