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

fix: PR template comments triggers bugfix label #60529

Merged
merged 1 commit into from
Aug 29, 2022

Conversation

casswedson
Copy link
Contributor

Summary

None

Purpose of change

#60287 makes pretty much all prs to be labeled as bugfixes, turns out the comments in the pr template match the regex needed to trigger the labeling

make sure the comments in the template don't result in a match for
the bugfix label

everything is not a bugfix

Describe the solution

Describe alternatives you've considered

Testing

go to the website where I tested the regex https://regex101.com/r/pBIUNe/1 and input the modified pr template, no matches

this pr also should not get the bugfix label

Additional context

dirty hack, change numbers to a place holder xxxx

make sure the comments in the template don't result in a match for
the bugfix label

everything is not a bugfix
@github-actions github-actions bot added Code: Tooling Tooling that is not part of the main game but is part of the repo. astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Aug 28, 2022
@dseguin dseguin merged commit 8f89856 into CleverRaven:master Aug 29, 2022
@casswedson casswedson deleted the everything-is-a-not-fix branch August 29, 2022 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Code: Tooling Tooling that is not part of the main game but is part of the repo. json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants