Skip to content

Commit 488ffe1

Browse files
Merge pull request #184 from NREL/PRTemplate
add pull request template
2 parents 123c745 + b79c0a0 commit 488ffe1

File tree

1 file changed

+37
-0
lines changed

1 file changed

+37
-0
lines changed

.github/pull_request_template.md

+37
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
## Pull Request Template
2+
3+
### Description
4+
5+
[ description of the bug fix or new functionality ]
6+
7+
[ include directions for reviewers if necessary- what or how should they test or review ]
8+
9+
Fixes # (issue(s))
10+
11+
### Corresponding branches and PRs:
12+
13+
[ which branches of wex, lk, and ssc should be built with this PR ]
14+
15+
[ link any corresponding PRs in other repos, i.e. NREL/ssc#x ]
16+
17+
### Unit Test Impact:
18+
19+
[ new tests written? ]
20+
21+
[ expected changes in unit tests or speed of tests? ]
22+
23+
[ expected changes in test_results files? ]
24+
25+
### Checklist
26+
- [ ] requires help revision and I added that label
27+
- [ ] adds, removes, modifies, or deletes variables in existing compute modules
28+
- [ ] adds a new compute module
29+
- [ ] changes defaults
30+
- [ ] I've tagged this PR to a milestone
31+
32+
### Reminders- this section can be deleted
33+
[Checking for PySAM Incompatible API Changes]
34+
(https://github.com/NREL/SAM/wiki/PySAM-Incompatible-API-Changes-&-Regenerating-PySAM-Files).
35+
36+
[When do the PySAM files need to be regenerated?]
37+
(https://github.com/NREL/SAM/wiki/PySAM-Incompatible-API-Changes-&-Regenerating-PySAM-Files#when-do-the-pysam-files-need-to-be-regenerated-via-export_config)

0 commit comments

Comments
 (0)