-
-
Notifications
You must be signed in to change notification settings - Fork 8
fix autolabeler by adding labeler.yml action #87
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
Conversation
This comment has been minimized.
This comment has been minimized.
expect docs CI failed, the autolabeler works as expected. I also add an input parameter |
TL;DR We need to use a different action to generate labels in a PR. The issue you linked to has diverged into 2 issues.
But there is currently no way differentiate which feature of release-drafter you want to use in specified event trigger. Using the |
Per your comments, I added a new action to autolabeler. From my test, the event |
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'm surprised that this works without errors. It seems to use the commitish
advice (in reusable workflow), but the labels (ie "bug" in this PR) are still assigned to the PR properly.
Co-authored-by: Brendan <2bndy5@gmail.com>
Co-authored-by: Brendan <2bndy5@gmail.com>
It would be work to keep or remove commitish from reusable workflow. |
Is that why |
I think so. commitish has default value in reusable workflow, it can be overwrite when the default branch is not main, for example: master |
Per as cpp-linter/.github#21