-
Notifications
You must be signed in to change notification settings - Fork 22
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
MTV-1883: Adding PR template #1434
Conversation
Signed-off-by: Matan Schatzman <mschatzm@redhat.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1434 +/- ##
==========================================
- Coverage 36.81% 36.20% -0.62%
==========================================
Files 158 159 +1
Lines 2548 2580 +32
Branches 599 615 +16
==========================================
- Hits 938 934 -4
- Misses 1428 1451 +23
- Partials 182 195 +13 ☔ View full report in Codecov by Sentry. |
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
I just suggest to change the sections order since it's more convenient for me to search for the bug reference first with the "References:"
header,
then description and screenshots (which can be long)
and CC at the end, if needed.
Co-authored-by: Sharon Gratch <sgratch@redhat.com> Signed-off-by: Matan Schatzman <matanschatzman@gmail.com>
Signed-off-by: Matan Schatzman <matanschatzman@gmail.com>
Quality Gate passedIssues Measures |
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
MTV-1883: Adding PR template
https://issues.redhat.com/browse/MTV-1883