-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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 Labeler Actions #1074
Add Labeler Actions #1074
Conversation
this will label PRs such that the maintainers have visibility into the breadth of the changes being suggested
.github/labeler.yml
Outdated
"area/lib": ./*.go | ||
|
||
# changes to the zsh completion | ||
"area/*sh completion": ./zsh* |
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.
Can we match on the bash completion as well?
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.
👍
just added the matchers for the other autocompletion shells
steps: | ||
- uses: actions/labeler@v2 | ||
with: | ||
repo-token: "${{ secrets.GITHUB_TOKEN }}" |
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.
Is this secret something I need to configure?
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.
I don't believe you need to do anything no.
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.
okay perfect
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.
FWIW, from #952 (comment):
Better integration of permissions. Since Actions is built in GitHub, it is possible to use secret tokens without explicitly managing/setting them.
By the same token, as suggested in #959, it should be possible to parse the body of PRs to have them self-labelled by contributors.
any changes to and autocomplete for any shell will now be auto labelled as impacting the autocompletion functionality
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 LGTM
|
||
# changes to the core lib package | ||
"area/lib": ./*.go | ||
|
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.
What about one for the Cobra Generator itself?
# changes to the cobra generator
"area/cobra-generator": cobra/**/*
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.
I didn't see that label in the current set.
Maybe we have a enhancement PR to add more labels and matchers?
@jpmcb
adds labeling actions
when certain parts of the code are modified by a
PR then the corresponding label is added