New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Don't load .env files by default #911
Conversation
Loading the `.env` files is a change in behaviour and broke backward-compatibility. It sounds really nice at first but it turns out that many users have .env files littered in their repository and cause a direnv reload. It's possible to turn that feature off but now they have one extra step to instruct their users. This commit inverts the behaviour and turns that feature off by default to restore back-compat. See also direnv#878.
Great! |
please also highlight in the PR body that the config key changed, it's not that visible. as both words are of the same length and the common part exceeds the differing part. so typical eye will optimize the words to be the same. |
I think the document misses the part where is the toml config, how to create it, and what to configure to use dotenv files back again. (Or I didn't find the info :p) Edit: Found out there https://direnv.net/man/direnv.toml.1.html |
A recent release of direnv disabled loading of .env files by default. See direnv/direnv#911. Loading of .env files can still be enabled by changing settings, but it's simpler to provide documentation and an example that just works without further changes beyond adding your personal settings / secrets.
* docs: update to work out-of-the-box with current direnv version A recent release of direnv disabled loading of .env files by default. See direnv/direnv#911. Loading of .env files can still be enabled by changing settings, but it's simpler to provide documentation and an example that just works without further changes beyond adding your personal settings / secrets. * docs: additional README updates
Follow-up on direnv#845 and direnv#911: resolves direnv#916 This commit simply updates the error messages and documentation to more accurately reflect the current opt-in configurable handling of `.env` files.
Loading the
.env
files is a change in behaviour and brokebackward-compatibility. It sounds really nice at first but it turns out
that many users have .env files littered in their repository and cause a
direnv reload. It's possible to turn that feature off but now they have
one extra step to instruct their users.
This commit inverts the behaviour and turns that feature off by default
to restore back-compat.
Before:
Now (with a different key name):
See also #878.