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
ci: update ngbot config file #22173
ci: update ngbot config file #22173
Conversation
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.
lgtm
@@ -50,13 +50,15 @@ merge: | |||
noConflict: true | |||
# list of labels that a PR needs to have, checked with a regexp (e.g. "PR target:" will work for the label "PR target: master") | |||
requiredLabels: | |||
- "PR target:" | |||
- "PR target: *" |
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 hope these are globs and not some custom patterns/matching language.
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.
The comment above says they are used as RegExps. I hope that's not true either 😃
It was using regexps previously but I've just updated the code to use globs
instead since it makes more sense, I'll update the comments
Le lun. 12 févr. 2018 23:20, George Kalpakas <notifications@github.com> a
écrit :
… ***@***.**** commented on this pull request.
------------------------------
In .github/angular-robot.yml
<#22173 (comment)>:
> @@ -50,13 +50,15 @@ merge:
noConflict: true
# list of labels that a PR needs to have, checked with a regexp (e.g. "PR target:" will work for the label "PR target: master")
requiredLabels:
- - "PR target:"
+ - "PR target: *"
The comment above says they are used as RegExps. I hope that's not true
either 😃
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#22173 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAQMos7roMCuqKT1-XuYcHiwVUUvrT2Fks5tUTexgaJpZM4SCnuJ>
.
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Some
type
labels aren't considered as triaged.Some PR state/action aren't blocking the merge when they should.
Issue Number: #22053
What is the new behavior?
Type refactor and type RFC/Discussion/question are now considered as triaged.
The labels PR action review and PR state blocked are now forbidden labels that will prevent the merge status to go green.
I've also slightly updated some regexp to be compliant with minimatch instead of just simple regexp. It should be easier to understand like that.
Does this PR introduce a breaking change?