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

pipx inject: Imply --include-apps if --include-deps is passed #984

Merged
merged 3 commits into from
Nov 30, 2023

Conversation

chrysle
Copy link
Contributor

@chrysle chrysle commented May 18, 2023

  • I have added an entry to docs/changelog.md

Summary of changes

Fixes #805

Test plan

Tested by running

pipx inject --include-deps ansible-core ansible-lint yamllint
nox -s tests-3.8

@chrysle chrysle force-pushed the inject-include-deps-imply-include-apps branch from 35409a6 to 3f25f27 Compare May 18, 2023 12:46
@chrysle chrysle changed the title pipx inject: Imply --include-apps if include--deps is passed pipx inject: Imply --include-apps if --include-deps is passed May 18, 2023
@gaborbernat gaborbernat merged commit 1508f18 into pypa:main Nov 30, 2023
0 of 4 checks passed
@chrysle chrysle deleted the inject-include-deps-imply-include-apps branch December 1, 2023 17:17
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

Successfully merging this pull request may close these issues.

Cannot pass --include-deps if --use_binaries is not passed as well
2 participants