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
meta: migrate labelling to actions (Discuss / WIP) #52695
base: main
Are you sure you want to change the base?
Conversation
Review requested:
|
- name: Initialize Pull Request | ||
uses: redyetidev/node-actions-runner@v0.0.1 | ||
with: | ||
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.
Could this be GH_USER_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.
Forks don't have access to GH_USER_TOKEN
, so no (and let's not go the pull_request_target way, it's not worth it because of the bad DX and the security aspects).
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.
Labelling was already going through a GitHub action under the nodejs org: https://github.com/nodejs/node-pr-labeler
Yes, I'm aware. I figured I'd consolidate it into a single workflow while changing the |
This pull request transitions the webhook and GitHub bot labeling and pinging system to a GitHub action, which I've configured at https://github.com/RedYetiDev/node-actions-runner/tree/main. However, this change has yet to be tested with the Node.js infrastructure, so it's a work in progress.
(This repo could also be moved to an internal one if need be)
Related to #51236
@nodejs/actions
@nodejs/github-bot