Improve compatibility with dotenv about prax.cr HOT 8 CLOSED

It sounds legit to drop .env now that we have .praxrc. I still like .env better, because it's more generic across tools, but different tools allow different parsing... sigh. Maybe we could interpolate variables as they are read from .env, looking in just parsed variables, then falling back to ENV?

from prax.cr.

Regarding interpolation: You could support it if you want. It wouldn't matter much to me, as long as Prax at least supported simple interpolation like VAR=$OTHER_VAR.

Although, for full compatibility with dotenv, Prax would also want to support other shell-isms, including optional export, optional single or double quotes, optional {} (like ${VARIABLE}), and escaping the $ with \ (like STATEMENT=SHOW ME THE \$\$\$). (See dotenv/parser.rb and dotenv/variable.rb.)

Personally, I wouldn't do it, as it seems to beckon maintaining a miniature embedded shell language.

from prax.cr.

Sigh, .env should be a simple A=B mecanism for providing the app with some per-install values, and then let the app interpolate the values, if necessary, from those bits...

from prax.cr.

Yep, this situation stinks. Another solution might be to keep the current ".env" parsing behavior, but allow the name of the file to be configured, so it could still be utilized, but not conflict with dotenv.

from prax.cr.

Another idea for implementing this: We rip the bandage off. We can immediately drop support for ".env", but if a ".env" file is detected in a project, Prax will print "Setting environment variables via '.env' has been removed. Please use a '.praxrc' shell script in the project to set environment variables instead. Disable this message by setting PRAX_ACKNOWLEDGE_ENV_REMOVAL=true."

I figure this may be better than deprecation and eventual removal, since we can immediately save dotenv users from the same fate that I had, also the removal is going to inconvenience people at some point no matter what (some people will wait to deal with deprecation until the bitter end).

from prax.cr.

Sweet, we finally fixed it! 🤝 Can we make a release?

from prax.cr.

Yes: v0.7.0

from prax.cr.

Installed the .deb yesterday and it worked just dandy.

from prax.cr.