-
Notifications
You must be signed in to change notification settings - Fork 364
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 only-labels option #22
Conversation
@@ -40,6 +41,7 @@ async function processIssues( | |||
owner: github.context.repo.owner, | |||
repo: github.context.repo.repo, | |||
state: 'open', | |||
labels: args.onlyLabels, |
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.
This can be comma-separated list of labels:
https://octokit.github.io/rest.js/#octokit-routes-issues
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.
Mention that in the description!
Any progress on this? |
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.
LGTM but just improve the description
Looking forward to this. @yuya-takeyama are you okay to accept the suggested change so that this can be merged? |
I'm really looking forward to this also. @karl-johan-grahn Can you please merge it, and add a commit on top of it to modify the description, if it's the only blocking thing? I can even do the commit in a second PR if you want me to. |
@yuya-takeyama Has to update the description and merge it, I cannot merge |
@chrispat I'm seeing that you merged the latest PR, can you help? |
Co-Authored-By: Stephan Miehe <49914758+stephanmiehe@users.noreply.github.com>
Thank you very much @hross! |
This is to address #18