Skip to content
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

Disable labelling of PRs as it is replaced by GitHub Action #296

Merged
merged 1 commit into from Apr 24, 2021

Conversation

phillipj
Copy link
Member

@phillipj phillipj commented Apr 22, 2021

First step of removing labelling of PRs as they are opened.

Only the part actively listening for newly opened PRs gets removed now, with the main goal of stopping @nodejs-github-bot from applying labels and thereby leaving room for nodejs/node-pr-labeler GitHub Action to apply those labels instead.

Still a lot of code that can be removed, like the actually logic of resolving labels which has been extracted into the said GitHub Action, but postponing that to a followup PR, when seeing the Action behaving as expected -- if not, these changes might be reverted as a fallback.

Refs nodejs/node#38301

First step of removing labelling of PRs as they are opened. Only the
part actively listening for newly opened PRs gets removed now, with the
main goal of *stopping* @nodejs-github-bot from applying labels and
thereby leaving room for nodejs/node-pr-labeler GitHub Action to apply
those labels intead.

Still a lot of code that can be removed, like the actually logic of
resolving labels which has been extracted into the said GitHub Action,
but postponing that to a followup PR, when seeing the Action behaving
as expected -- if not, these changes might be reverted as a fallback.

Refs https://github.com/nodejs/node-pr-labeler
@phillipj phillipj requested a review from a team as a code owner April 22, 2021 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants