Skip to content
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

"Loading .env environment variables" logs to stderr #2358

Closed
caspervdw opened this issue Jun 14, 2018 · 7 comments
Closed

"Loading .env environment variables" logs to stderr #2358

caspervdw opened this issue Jun 14, 2018 · 7 comments

Comments

@caspervdw
Copy link

This is a minor annoyance, as in some scripts we log stderr.
I think this should be changed, as it is not an error message.

Thanks in advance!

err=True,

@techalchemy
Copy link
Member

Hey, thanks for the input! I agree it isn’t an error, but we actually log most things to stderr for various reasons. If we redo this it will be part of a broad restructuring, i would be very hesitant to change where we send messages haphazardly. Thanks again though!

@uranusjr
Copy link
Member

Maybe it’d be reasonable to add an environ (PIPENV_QUIET) or something to suppress some of the informational messages we produce. This, the “installing initially failing packages”, the “ignoring marker”, and probably some else.

@techalchemy
Copy link
Member

I think we have an env var and we just don't really use it much. We should flag this as a tracking item in a cleanup project

@foozmeat
Copy link

foozmeat commented Jan 2, 2019

@techalchemy apologies for replying in a closed issue but I didn't want to open a dupe to add my thoughts.

I currently use pipenv run in several cron jobs and this creates a situation where I have to send all output to /dev/null or I get an email with Loading .env environment variables… on each run. I'm running the risk of not seeing real error output because I can't silence this message. I'd be all for a -q quiet flag or PIPENV_QUIET to handle this case.

Your last comment seems to indicate that you meant to re-open this. Maybe?

@uranusjr
Copy link
Member

uranusjr commented Jan 3, 2019

The PIPENV_QUIET environment variable has since been implemented, but it is not applied to run yet. Feel free to work on this.

@foozmeat
Copy link

foozmeat commented Jan 3, 2019

@uranusjr I'm surprised to read that it has been implemented. As of version 2018.11.26 it doesn't appear in the list shown by pipenv --envs or listed in the docs.

I'll take a look at the code and try to implement it.

@thunderk
Copy link

I proposed a fix in #3457, if someone wants to reconsider this issue and review the PR. I think this issue is important for pipenv adoption, as properly using pipenv run commands in cron may be a frequent use case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants