Skip to content

Commit 0a08851

Browse files
Update self-review checklist and potentially add to PR template (github#19084)
* Update contributing * Add line to PR template * Apply suggestions from code review Co-authored-by: Dominik Bamberger <72066+b4mboo@users.noreply.github.com> * Update PR template Co-authored-by: Dominik Bamberger <72066+b4mboo@users.noreply.github.com>
1 parent 91d4f62 commit 0a08851

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,8 @@ Thanks again!
2222
<!-- Share artifacts of the changes, be they code snippets, GIFs or screenshots; whatever shares the most context. -->
2323

2424
### Check off the following:
25-
- [ ] I have reviewed my changes in staging. (look for the **deploy-to-heroku** link in your pull request, then click **View deployment**)
26-
- [ ] For content changes, I have reviewed the [localization checklist](https://github.com/github/docs/blob/main/contributing/localization-checklist.md)
27-
- [ ] For content changes, I have reviewed the [Content style guide for GitHub Docs](https://github.com/github/docs/blob/main/contributing/content-style-guide.md).
25+
- [ ] I have reviewed my changes in staging (look for the **deploy-to-heroku** link in your pull request, then click **View deployment**).
26+
- [ ] For content changes, I have completed the [self-review checklist](https://github.com/github/docs/blob/main/CONTRIBUTING.md#self-review).
2827

2928
### Writer impact (This section is for GitHub staff members only):
3029

CONTRIBUTING.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -158,12 +158,12 @@ We (usually the docs team, but sometimes GitHub product managers, engineers, or
158158
You should always review your own PR first.
159159

160160
For content changes, make sure that you:
161-
- [ ] Confirm that the changes address every part of the content design plan from your issue (if there are differences, explain them).
161+
- [ ] Confirm that the changes meet the user experience and goals outlined in the content design plan (if there is one).
162+
- [ ] Compare your pull request's source changes to staging to confirm that the output matches the source and that everything is rendering as expected. This helps spot issues like typos, content that doesn't follow the style guide, or content that isn't rendering due to versioning problems. Remember that lists and tables can be tricky.
162163
- [ ] Review the content for technical accuracy.
163164
- [ ] Review the entire pull request using the [localization checklist](contributing/localization-checklist.md).
164-
- [ ] Copy-edit the changes for grammar, spelling, and adherence to the style guide.
165+
- [ ] Copy-edit the changes for grammar, spelling, and adherence to the [style guide](https://github.com/github/docs/blob/main/contributing/content-style-guide.md).
165166
- [ ] Check new or updated Liquid statements to confirm that versioning is correct.
166-
- [ ] Check that all of your changes render correctly in staging. Remember, that lists and tables can be tricky.
167167
- [ ] If there are any failing checks in your PR, troubleshoot them until they're all passing.
168168

169169
### Pull request template

0 commit comments

Comments
 (0)