Skip to content

Commit 084d262

Browse files
authored
Merge pull request TheThingsIndustries#136 from TheThingsIndustries/util/pr-template-screenshot
Add screenshot req to PR template
2 parents 42e921d + aa38246 commit 084d262

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,13 @@ Closes #0000, References #0000, etc.
1111

1212
...
1313

14+
#### Screenshots
15+
<!-- Post a screenshot of your rendered content
16+
NOTE: This section is optional.
17+
-->
18+
19+
...
20+
1421
#### Changes
1522
<!-- What are the changes made in this pull request? -->
1623

@@ -33,7 +40,7 @@ implementation proposal in the referenced issues.
3340
<!-- Make sure that this pull request is complete. -->
3441

3542
- [ ] Scope: The referenced issue is addressed, there are no unrelated changes.
36-
- [ ] Run Locally: Verified that the docs build using `make server`
43+
- [ ] Run Locally: Verified that the docs build using `make server`, posted screenshots, verified external links.
3744
- [ ] New Features Marked: Documentation for new features is marked using the `new-in-version` shortcode, according to the guidelines in [CONTRIBUTING](CONTRIBUTING.md).
3845
- [ ] Style Guidelines: Documentation obeys style guidelines in [CONTRIBUTING](CONTRIBUTING.md).
3946
- [ ] Commits: Commit messages follow guidelines in [CONTRIBUTING](CONTRIBUTING.md), there are no fixup commits left.

0 commit comments

Comments
 (0)