Skip to content

Fix formatting in PR template checklist - #5024

Merged
copybara-service[bot] merged 1 commit into
google:mainfrom
rapsealk:ci/pr-template-line-feeds
Oct 15, 2025
Merged

copybara-service[bot] merged 1 commit into
google:mainfrom
rapsealk:ci/pr-template-line-feeds

Conversation

@rapsealk

Copy link
Copy Markdown
Contributor

What does this PR do?

Follow-up of ab90975.

This pull request makes a small change to the .github/pull_request_template.md file, improving the formatting of the checklist section by consolidating multiline checklist items into single lines for better readability.

I found the checklist in current PR template is somewhat broken as below:

Checklist

  • This PR fixes a minor issue (e.g.: typo or small bug) or improves the docs (you can dismiss the other
    checks if that's the case).
  • This change is discussed in a Github issue/
    discussion (please add a
    link).
  • The documentation and docstrings adhere to the
    documentation guidelines.
  • This change includes necessary high-coverage tests.
    (No quality testing = no merge!)

@rapsealk

Copy link
Copy Markdown
Contributor Author

Was it meant to improve readability?

@vfdev-5

vfdev-5 commented Oct 13, 2025

Copy link
Copy Markdown
Collaborator

It was created 3 years ago. We can make each line as a single line. I have no strong opinion.

@cgarciae cgarciae left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @rapsealk !

@copybara-service
copybara-service Bot merged commit bc8d18a into google:main Oct 15, 2025
15 checks passed
@rapsealk
rapsealk deleted the ci/pr-template-line-feeds branch October 16, 2025 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants