Skip to content

[Console] Allow short command "p" on ProcessCommand#7931

Open
samsonasik wants to merge 6 commits intomainfrom
allow-short-command-p
Open

[Console] Allow short command "p" on ProcessCommand#7931
samsonasik wants to merge 6 commits intomainfrom
allow-short-command-p

Conversation

@samsonasik
Copy link
Member

@samsonasik samsonasik commented Mar 16, 2026

with add e2e test for it.

Fixes rectorphp/rector#9693

@samsonasik samsonasik requested a review from TomasVotruba March 16, 2026 21:23
@samsonasik
Copy link
Member Author

@TomasVotruba ready 👍

@samsonasik
Copy link
Member Author

synced latest main branch 👍 , @TomasVotruba ready to merge 👍

@samsonasik
Copy link
Member Author

It seems the real solution is by using setAliases(['p']) on ProcessCommand 👍 41447c0

Ready to merge 👍

@samsonasik samsonasik changed the title [Console] Allow short command "p" on ConsoleApplication [Console] Allow short command "p" on ProcessCommand Mar 17, 2026
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.

Since 2.3.9, short command name is broken

2 participants