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 git prs as an alias for git pull-requests #64

Merged
merged 1 commit into from Sep 21, 2022
Merged

Conversation

benchristel
Copy link
Member

Summary:

https://github.com/Khan/our-lovely-cli/pull/530 added a git
pull-requests command to OLC. This adds an alias for it that's easier to
type.

Issue: none

Test plan:

Run git prs. You should see a report of your outstanding pull
requests.

Khan/our-lovely-cli#530 added a git
pull-requests command to OLC. This adds an alias for it that's easier to
type.

Issue: none

Test plan:

Run `git prs`. You should see a report of your outstanding pull
requests.
@@ -27,6 +27,7 @@
rb = !git review-branch
pb = !git review-branch ; short for 'phabricator branch'
pr = !git pull-request
prs = !git pull-requests
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TIL about the ! prefix for git aliases. https://git-scm.com/book/en/v2/Git-Basics-Git-Aliases

Copy link
Member

@lillialexis lillialexis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Before you land this, I would double-check that this command (and– I'm realizing totally too late– the git pull-requests command) doesn't collide with GitHub's official CLI, gh. That might also have "porcelain" commands pull-requests and prs, and if a user has both installed, one or the other will trample the other. (Based on order in the _PATH_, I believe.

@benchristel
Copy link
Member Author

Ooh, good call. Looks like gh just has pr as a subcommand, with further subcommands treed off of that. So I think this is safe.

@benchristel benchristel merged commit 830604f into master Sep 21, 2022
@benchristel benchristel deleted the benc/git-prs branch September 21, 2022 18:33
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.

None yet

2 participants