-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 pull request template #5545
update pull request template #5545
Conversation
Signed-off-by: svrnm <[email protected]>
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.
Rewrote a couple parts, but LGTM.
Co-authored-by: Tiffany Hrabusa <[email protected]>
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.
Whatever suggestions you choose to adopt in #5544 (review) should be incorporated here too.
Signed-off-by: svrnm <[email protected]>
/fix:format |
You triggered fix:format action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/11785602735 |
IMPORTANT: (RE-)RUN
|
Will take this one in as-is, it looks good and is an improvement over the other. |
Signed-off-by: svrnm <[email protected]> Co-authored-by: Tiffany Hrabusa <[email protected]> Co-authored-by: opentelemetrybot <[email protected]> Co-authored-by: Phillip Carter <[email protected]>
Updates the pull request template to ask the contributor for sharing some details on their PR. It also let's the contributor know about CI checks, how they can address potential issues and that it is finally not their responsibility to have them fixed perfectly.