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

Always read labelling config from the master branch #9

Merged
merged 1 commit into from Apr 26, 2021

Conversation

phillipj
Copy link
Member

Instead of reading the labelling config from the target branch/SHA, which for the nodejs/node project might be a staging branch.

As observed by @targos, always having to cherry-pick labelling config changes merged to the master branch, into different staging branches is very inconvenient and will be forgotten from time to time.

Let's instead always read the labelling config from the master branch.

Refs nodejs/node#38301 (comment)

/cc @nodejs/github-bot

Instead of reading the labelling config from the target branch/SHA,
which for the nodejs/node project might be a staging branch.

Always having to cherry-pick labelling config changes merged to the
master branch, into different staging branches is very inconvenient
and will be forgotten from time to time.
@phillipj phillipj merged commit fe83a8b into main Apr 26, 2021
@phillipj phillipj deleted the always-config-from-master branch April 26, 2021 13:09
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