Commit 418de19
authored
[Documentation:Developer] Add PR Template Warning (#12342)
### Why is this Change Important & Necessary?
Recently, we have been getting an influx of pull requests that do not
comply with Submitty's conventions. This leads to a drawn out review
process as we have to wait for contributors to update their PR before we
can even review it.
### What is the New Behavior?
Added the following to the top of our pull request description template:
```
<!-- ** Please remove all comment blocks in the description before submitting this PR. ** -->
<!-- NOTE: Please ensure your title and description align with Submitty conventions (see
https://submitty.org/developer/getting_started/make_a_pull_request for more details).
Each title has a prefix, and a limit of 40 chars. A description template has been
provided and must be completed in full. Please also ensure that all CI tests
are passing. Pull requests that do not meet these requirements are ineligible for
review and may be closed. -->
```
### Automated Testing & Documentation
We should add more documentation tests that check if comments have been
deleted, and verify that the template headers are within the
description.
### What steps should a reviewer take to reproduce or test the bug or
new feature?
Is this change sufficient? Should I add/remove anything?1 parent 81c3909 commit 418de19
1 file changed
+9
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
1 | 10 | | |
2 | 11 | | |
3 | 12 | | |
| |||
0 commit comments