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
Hide unnecessary information in the rendered PR template #11383
Conversation
@@ -1,10 +1,13 @@ | |||
Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: | |||
Replace this paragraph with a description of the changes, then follow the instructions in the next section. |
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.
Maybe this should also be a comment that just says "Enter a description of your changes below this line:" (and then some blank space).
(I think that makes it a little harder to gloss over this step.)
Same comment as #11340 (comment) - I find that people mostly add their own content at the end of the template instead of at the top. |
@aneeshusa While I can believe that, we may be able to influence that decision. I think doing so is useful, since it will address one of @nox's common complaints about the new template. |
Works for me, although mbrubeck's suggested change would make it much more natural (having to replace existing text is just weird). |
This looks good to me. I think @mbrubeck's suggestion is a good one, but I'm happy to try it this way first if you prefer that. If we see anyone submitting PRs without changing this text, then I think we should definitely make that change. This is r=me but I didn't r+ it since mbrubeck is the assignee and his suggestion was unaddressed in the comments. |
New code was committed to pull request. |
@mbrubeck See what you think of my newest text. |
@bors-servo r+ |
📌 Commit 76cba2a has been approved by |
@bors-servo p=-1 |
☔ The latest upstream changes (presumably #11409) made this pull request unmergeable. Please resolve the merge conflicts. |
OMG. How sad that we can bitrot this :) |
New code was committed to pull request. |
@bors-servo: r=mbrubeck |
📌 Commit 1cfc2c9 has been approved by |
@bors-servo: p=-1 |
Hide unnecessary information in the rendered PR template These changes attempt to make the initial PR view less cluttered. Instructions are hidden in HTML comments, and there's a clear distinction between the PR description and the checklist. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data:--> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy --faster` does not report any errors - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because not part of the build <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11383) <!-- Reviewable:end -->
💔 Test failed - mac-rel-css |
@bors-servo retry mac1 |
Hide unnecessary information in the rendered PR template These changes attempt to make the initial PR view less cluttered. Instructions are hidden in HTML comments, and there's a clear distinction between the PR description and the checklist. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data:--> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy --faster` does not report any errors - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because not part of the build <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11383) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows |
These changes attempt to make the initial PR view less cluttered. Instructions are hidden in HTML comments, and there's a clear distinction between the PR description and the checklist.
./mach build -d
does not report any errors./mach test-tidy --faster
does not report any errorsThis change is