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

Add uninstall as alias to composer remove. #11951

Merged
merged 1 commit into from Apr 29, 2024
Merged

Conversation

BusterNeece
Copy link
Contributor

I've looked through the repository and realized I've never actually seen this proposed either in issue or pull request form.

Some package managers use uninstall as the convention for removing/uninstalling dependencies, and as a result I've found myself in the habit of typing composer uninstall (packagename) instead of composer remove (packagename).

Composer doesn't currently handle this all that gracefully...it suggests "Did you mean install/reinstall?" without suggesting the actual proper remove command.

I think, perhaps to save folks like me the trouble, Composer can follow in the footsteps of package managers like NPM and make remove and uninstall refer to the same command, so there's no retyping involved if you slip up and forget which is which.

@Seldaek Seldaek merged commit 7c66169 into composer:main Apr 29, 2024
6 of 20 checks passed
@Seldaek Seldaek added this to the 2.7 milestone Apr 29, 2024
@Seldaek
Copy link
Member

Seldaek commented Apr 29, 2024

Sure why not, thanks

@SamiQu
Copy link

SamiQu commented May 7, 2024

Ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants