Skip to content

Commit

Permalink
Merge pull request #2066 from terrapower/main
Browse files Browse the repository at this point in the history
Fix `pip-compile` to properly exclude non-relevant options from output header
  • Loading branch information
chrysle authored Mar 11, 2024
2 parents 60ebdf5 + ed0e64c commit 4f44a02
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
7 changes: 6 additions & 1 deletion piptools/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -371,9 +371,14 @@ def get_compile_command(click_ctx: click.Context) -> str:
# Get the latest option name (usually it'll be a long name)
option_long_name = option.opts[-1]

negative_option = None
if option.is_flag and option.secondary_opts:
# get inverse flag --no-{option_long_name}
negative_option = option.secondary_opts[-1]

# Exclude one-off options (--upgrade/--upgrade-package/--rebuild/...)
# or options that don't change compile behaviour (--verbose/--dry-run/...)
if option_long_name in COMPILE_EXCLUDE_OPTIONS:
if {option_long_name, negative_option} & COMPILE_EXCLUDE_OPTIONS:
continue

# Exclude config option if it's the default one
Expand Down
2 changes: 2 additions & 0 deletions tests/test_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -374,6 +374,8 @@ def test_is_url_requirement_filename(caplog, from_line, line):
(["--upgrade"], "pip-compile"),
(["-P", "django"], "pip-compile"),
(["--upgrade-package", "django"], "pip-compile"),
(["--reuse-hashes"], "pip-compile"),
(["--no-reuse-hashes"], "pip-compile"),
# Check options
(["--max-rounds", "42"], "pip-compile --max-rounds=42"),
(["--index-url", "https://foo"], "pip-compile --index-url=https://foo"),
Expand Down

0 comments on commit 4f44a02

Please sign in to comment.