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

Fix lock -r warning being logged to stdout instead of stderr #5091

Merged
merged 3 commits into from
May 2, 2022
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 6 additions & 7 deletions pipenv/cli/command.py
Original file line number Diff line number Diff line change
Expand Up @@ -324,14 +324,13 @@ def lock(ctx, state, **kwargs):
pre = state.installstate.pre
if emit_requirements:
secho(
"""
Warning: The lock flag -r/--requirements will be deprecated in a future version
of pipenv in favor of the new requirements command. For more info see
https://pipenv.pypa.io/en/latest/advanced/#generating-a-requirements-txt
NOTE: the requirements command parses Pipfile.lock directly without performing any
locking operations. Updating packages should be done by running pipenv lock
""",
"Warning: The lock flag -r/--requirements will be deprecated in a future version\n"
"of pipenv in favor of the new requirements command. For more info see\n"
"https://pipenv.pypa.io/en/latest/advanced/#generating-a-requirements-txt\n"
"NOTE: the requirements command parses Pipfile.lock directly without performing any\n"
"locking operations. Updating packages should be done by running pipenv lock.",
fg="yellow",
err=True,
)
# Emit requirements file header (unless turned off with --no-header)
if state.lockoptions.emit_requirements_header:
Expand Down