-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Update bot for SmallRye Metrics, Micrometer, Tracing #16474
Conversation
.github/quarkus-bot.yml
Outdated
@@ -127,24 +127,33 @@ triage: | |||
- extensions/vertx-graphql/ | |||
- integration-tests/vertx-graphql/ | |||
- labels: [area/tracing, area/smallrye] | |||
title: "(trace|tracing|telemetry)" | |||
titleBody: "(trace|tracing)" |
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 strongly suggest you don't do that. You will have a ton of false positives. I removed all the titleBody
because it pinged people for nothing. Typically when people include the list of their extensions in the issue or when they use the word trace or tracing for another purpose.
Not doing that gives us a bit more triaging job but not that much: people usually include what's not working in the title.
Same comments for all the titleBody
s below.
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.
No worries, I switched it back
Failing Jobs - Building 4216db9
Full information is available in the Build summary check run. Test Failures⚙️ JVM Tests - JDK 11 #📦 integration-tests/kafka✖ ✖ |
No description provided.