-
Notifications
You must be signed in to change notification settings - Fork 55
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #1310 from NREL/pr_template_trd
Update PR template
- Loading branch information
Showing
1 changed file
with
17 additions
and
7 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,24 @@ | ||
## Pull Request Description | ||
|
||
[description here] | ||
|
||
### Related Pull Requests | ||
[related PRs from different repositories] | ||
|
||
### Related Issues | ||
[What issue(s) is the PR addressing] | ||
|
||
## Checklist | ||
|
||
Not all may apply: | ||
#### Required: | ||
- [ ] [Technical reference documentation](https://github.com/NREL/resstock/tree/develop/docs/read_the_docs) has been updated | ||
- [ ] Add to the [changelog_dev.rst file](https://github.com/NREL/resstock/tree/develop/docs/read_the_docs/source/changelog/changelog_dev.rst) | ||
- [ ] No unexpected regression test changes on CI (comparison artifacts have been checked) | ||
|
||
#### Optional (not all items may apply): | ||
- [ ] Tests (and test files) have been updated | ||
- [ ] Documentation has been updated | ||
- [ ] If related to resstock-estimation, checklist includes [data dictionary](https://github.com/NREL/resstock/tree/develop/resources/data/dictionary), [source report](https://github.com/NREL/resstock/tree/develop/project_national/resources/source_report.csv), [options saturation](https://github.com/NREL/resstock/tree/develop/project_national/resources/options_saturations.csv), [options_lookup](https://github.com/NREL/resstock/blob/develop/resources/options_lookup.tsv). | ||
- [ ] Add to the [changelog_dev.rst file](https://github.com/NREL/resstock/tree/develop/docs/read_the_docs/source/changelog/changelog_dev.rst) | ||
- [ ] `openstudio tasks.rb update_measures` has been run | ||
- [ ] No unexpected regression test changes on CI (checked comparison artifacts) | ||
- [ ] Supporting resource files have been updated (related to resstock-estimation) | ||
- [ ] [data dictionary](https://github.com/NREL/resstock/tree/develop/resources/data/dictionary) | ||
- [ ] [source report](https://github.com/NREL/resstock/tree/develop/project_national/resources/source_report.csv) | ||
- [ ] [options saturation](https://github.com/NREL/resstock/tree/develop/project_national/resources/options_saturations.csv) | ||
- [ ] [options_lookup](https://github.com/NREL/resstock/blob/develop/resources/options_lookup.tsv) | ||
- [ ] `openstudio tasks.rb update_measures` has been run |