Replies: 30 comments
-
A dockerimage would be nice I think |
Beta Was this translation helpful? Give feedback.
-
Could you give more information about that? Do you want to run this linter using a Docker image? |
Beta Was this translation helpful? Give feedback.
-
Yes, an image with an already compiled binary which I can just use without the need to install |
Beta Was this translation helpful? Give feedback.
-
Disabling/enabling comments. It's a good option, i think some users may want to disable some checkers # linter:disable UnorderedKeysChecker
MY=val
OWN=val
ORDER=val |
Beta Was this translation helpful? Give feedback.
-
I think autofix option is awesome, that's what i really love in linters LOGGER_LEVEL=info
RAILS_ENV
DB-NAME=testing
DEbUG_hTTP=true
DB_NAME=development
run
We'll get: DB_NAME=development
DEBUG_HTTP=true
LOGGER_LEVEL=info
RAILS_ENV= |
Beta Was this translation helpful? Give feedback.
-
Great idea! I've just created an issue to do that #34. |
Beta Was this translation helpful? Give feedback.
-
I think it's a good idea! Could you open an issue and write your vision on how this will work? |
Beta Was this translation helpful? Give feedback.
-
Awesome idea! Thank you! I will think about how to implement it. |
Beta Was this translation helpful? Give feedback.
-
Maybe it would be a good idea to give the user the ability to lint a specific file, i.e.: |
Beta Was this translation helpful? Give feedback.
-
I think it would be good to add check for lines trailing spaces, what do you think? Also it would be good to skip blank lines. Blank lines could be used in env files for separation some blocks of variables. ENV file like this: FOO=BAR
BAR=FOO prints out And I've created issue for filtering out comments #36 |
Beta Was this translation helpful? Give feedback.
-
You're right! I've already thought about it. I will open an issue to do that a bit later. |
Beta Was this translation helpful? Give feedback.
-
Good ideas! Thank you! |
Beta Was this translation helpful? Give feedback.
-
Change exit code to nonzero if some checks are failing – it will be useful for CI |
Beta Was this translation helpful? Give feedback.
-
Add ability to include files to check - #45 |
Beta Was this translation helpful? Give feedback.
-
Good catch! Thank you! #47 |
Beta Was this translation helpful? Give feedback.
-
I don't know exactly about the pros of this feature, but what do you think about creation |
Beta Was this translation helpful? Give feedback.
-
Thank you! |
Beta Was this translation helpful? Give feedback.
-
what about allowing to have a config file, where the excluded checks are defned? |
Beta Was this translation helpful? Give feedback.
-
Thanks, I think that's a good idea! |
Beta Was this translation helpful? Give feedback.
-
It might be out of the scope of this project but it would be great if your utility could also inject the values into the current shell/bash/cmd/etc session. It's really hard to import environment variables from a dotenv file to a OS in a way that is platform agnostic Something like:
|
Beta Was this translation helpful? Give feedback.
-
Hey @alshdavid, thank you for your idea but to be honest I think this is out of scope as you already mentioned. |
Beta Was this translation helpful? Give feedback.
-
great tool btw, thank you! it would be cool if it had the ability to diff across files to compare say |
Beta Was this translation helpful? Give feedback.
-
Thank you! We will think about it soon. |
Beta Was this translation helpful? Give feedback.
-
A cool idea, I really like that to be honest. |
Beta Was this translation helpful? Give feedback.
-
We use https://www.npmjs.com/package/dotenv-safe to make sure that any missing environment variables are flagged at runtime. It might be nice to have an option in this tool to check a |
Beta Was this translation helpful? Give feedback.
-
Thank you! It seems to me that @mstruebing has already opened a PR with similar functionality #282 🤔 |
Beta Was this translation helpful? Give feedback.
-
@mgrachev Thanks, that's great news! 👍 |
Beta Was this translation helpful? Give feedback.
-
We are happy to announce the release of dotenv-linter - v3.0.0 🥳 |
Beta Was this translation helpful? Give feedback.
-
Hey folks, crossposting from "modenv" project: kurtbuilds/modenv#4 Do you think However, I prefer having one go-to solution than multiple projects, so I could instead try to add this feature to This comment is also related, .env injection is a key feature of Doppler: #23 (comment) |
Beta Was this translation helpful? Give feedback.
-
I found out by accident that passing a directory as an the first argument to compare you can compare all .env files Windows:
Linux
Output:
Hope that helps someone |
Beta Was this translation helpful? Give feedback.
-
Hi there!
If you have any ideas on how to improve this project or what other checks we should add - welcome 😉
Beta Was this translation helpful? Give feedback.
All reactions