Skip to content

Add alias support for install and uninstall #11028

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

chloegoh
Copy link

As per pradyunsg's comment, I have added the i and u aliases based off of McSinyx's implementation at #8137.

In summary, this fixes issue #8130 and allows users to install packages with i and add, and uninstall packages with u and remove.

$ pip help
Usage:
  pip <command> [options]

Commands:
  install, add, i             Install packages.
  download                    Download packages.
  uninstall, remove, u        Uninstall packages.
...

$ pip i numpy      # or pip add numpy
Collecting numpy
  Using cached numpy-1.22.3-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (16.8 MB)
Installing collected packages: numpy
Successfully installed numpy-1.22.3

$ pip u numpy      # or pip remove numpy
Found existing installation: numpy 1.22.3
Uninstalling numpy-1.22.3:
  Would remove:
...

Credits to McSinyx for implementing the alias mechanism.

@uranusjr
Copy link
Member

Please fix linter errors.

@chloegoh
Copy link
Author

Hi, I noticed that when I run tests locally using nox, there are some OS error messages triggered in tests/unit/test_command_install.py. I initially thought that it was caused by my environment, but seems like the same issue occurs in the CI tests. Would appreciate any guidance on this!

@uranusjr
Copy link
Member

Could be (not sure) affected by https://github.blog/2022-04-12-git-security-vulnerability-announced/

@github-actions github-actions bot added the needs rebase or merge PR has conflicts with current master label Jan 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs rebase or merge PR has conflicts with current master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants