Skip to content

Commit c45a79f

Browse files
authored
Merge PR #135 by @Rulasmur - GitHub PR template
2 parents c0d96e2 + bacc835 commit c45a79f

File tree

1 file changed

+25
-0
lines changed

1 file changed

+25
-0
lines changed

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
<!-- Thanks for submitting a pull request for Destination Sol! :-)
2+
Please fill in some brief details below about the PR.
3+
If it contains source code please make sure to do everything in the pre pull request checklist first.
4+
If you add unit tests we'll love you forever! -->
5+
6+
# Description
7+
> Please describe what this PR does and why it does it
8+
9+
# Testing
10+
> If the PR is testable, please describe how to test it here
11+
12+
# Outstanding Work
13+
- [ ] If there is work still outstanding, please include it here
14+
- [ ] if there is nothing outstanding, remove this section.
15+
16+
# Notes
17+
> Include any extra notes here
18+
19+
### Pre Pull Request Checklist:
20+
<!-- When scanning the code with SonarLint, just don't introduce any new issues, and maybe even fix a few existing ones -->
21+
- [ ] Code has been scanned with [SonarLint](http://www.sonarlint.org/intellij/)
22+
- [ ] Methods have been annotated with @ Nullable and @ Nonnull ([Read more](https://github.com/MovingBlocks/DestinationSol/wiki/@Nullable-&-@Nonnull-Quickstart))
23+
- [ ] There are no errors present in the project
24+
- [ ] Code has been formatted and indented
25+
- [ ] Methods have appropriate Javadoc ([How to write Javadoc](http://www.oracle.com/technetwork/java/javase/documentation/index-137868.html))

0 commit comments

Comments
 (0)