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

feat: allow upgrade command to accept multiple packages as arguments #1339

Merged
merged 4 commits into from
Apr 12, 2024

Conversation

huxuan
Copy link
Member

@huxuan huxuan commented Apr 11, 2024

  • I have added a news fragment under changelog.d/ (if the patch affects the end users)

Summary of changes

Close #1336

Test plan

Tested by running

# command(s) to exercise these changes
nox -s tests

@huxuan huxuan marked this pull request as ready for review April 11, 2024 06:06
@huxuan huxuan requested a review from uranusjr April 11, 2024 10:53
src/pipx/main.py Outdated Show resolved Hide resolved
tests/test_upgrade.py Outdated Show resolved Hide resolved
tests/test_upgrade.py Outdated Show resolved Hide resolved
Co-authored-by: Jason Lam <meowmeowcat1211@gmail.com>
@huxuan
Copy link
Member Author

huxuan commented Apr 12, 2024

Hi @dukecat0,

Sorry, I just copied some code snippets from the surrounding test cases, I should be more careful.

Thanks for pointing out.

@uranusjr uranusjr merged commit 5482fac into pypa:main Apr 12, 2024
14 checks passed
@huxuan huxuan deleted the xuan.hu/upgrade-multiple branch April 12, 2024 06:10
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.

Allow pipx upgrade to accept multiple packages as arguments
4 participants