should `CurrentPackage()` be deprecated? - Githubissues

gobuffalo / envy

Envy makes working with ENV variables in Go trivial.

MIT License

156 stars 21 forks source link

should `CurrentPackage()` be deprecated? #37

Closed sio4 closed 2 years ago

sio4 commented 2 years ago

This implementation is based on the legacy layout of GOPATH which is deprecated now. It seems like CurrentModule() should be used for the same purpose. Any idea?

https://github.com/gobuffalo/envy/blob/3dbe4c98c1c77e4a8fac053251c262a85e048f21/envy.go#L218-L232

https://github.com/gobuffalo/envy/blob/3dbe4c98c1c77e4a8fac053251c262a85e048f21/envy.go#L254-L259