-
Notifications
You must be signed in to change notification settings - Fork 7.8k
Add gh browse --actions flag #12091
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 gh browse --actions flag #12091
Conversation
|
Hi! Thanks for the pull request. Please ensure that this change is linked to an issue by mentioning an issue number in the description of the pull request. If this pull request would close the issue, please put the word 'Fixes' before the issue number somewhere in the pull request body. If this is a tiny change like fixing a typo, feel free to ignore this message. |
f4212b2 to
e89ecc2
Compare
babakks
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR, @rneatherway! 🍻
LGTM, except for missing tests. Can you please add the tests? You can basically replicate the Wiki flag tests.
e89ecc2 to
786d276
Compare
No problem, done. |
babakks
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the prompt fix! 🍻
|
Merging as the failed workflow has nothing to do with this PR (it's about unfixed vulnerability in |
Description
Fixes #12249
I've wanted this a few times recently and it seems like a reasonable addition in keeping with the other flags.