-
Notifications
You must be signed in to change notification settings - Fork 7.8k
Closed
Labels
coreThis issue is not accepting PRs from outside contributorsThis issue is not accepting PRs from outside contributorsgh-prrelating to the gh pr commandrelating to the gh pr commandtech-debtA chore that addresses technical debtA chore that addresses technical debt
Description
Follow-up to #11057
As a follow-up to the temporary fix we made in #11057, we should do a proper fix by decoupling arg parsing from PR finder. @williammartin has already explained what needs to be done in the PR's description and the code.
Since this is a refactoring task, there should be no observable changes. Specifically, The affected/refactored pr subcommands (e.g. pr edit or pr view) must support PR URLs as argument.
Expected Outcomes
- Existing tests related to affected
prcommands will still pass
Metadata
Metadata
Assignees
Labels
coreThis issue is not accepting PRs from outside contributorsThis issue is not accepting PRs from outside contributorsgh-prrelating to the gh pr commandrelating to the gh pr commandtech-debtA chore that addresses technical debtA chore that addresses technical debt