Skip to content

Commit b13197a

Browse files
committed
Add pull request templates
1 parent 8b5e006 commit b13197a

File tree

4 files changed

+94
-0
lines changed

4 files changed

+94
-0
lines changed

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
Please click the `Preview` tab and select a PR template:
2+
3+
- [New benchmark](?expand=1&template=new_benchmark.md)
4+
- [New model](?expand=1&template=new_model.md)
5+
- [All other proposed changes](?expand=1&template=custom.md)
Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
<!--
2+
Thank you for contributing! To help us out with reviewing, please consider the following:
3+
4+
- Does this pull request include a summary of the change (see below)?
5+
- Does this pull request include a descriptive title?
6+
- Does this pull request link to an issue (see below)?
7+
-->
8+
9+
## Pre-review checklist for PR author
10+
11+
PR author **must** check the checkboxes below when creating the PR.
12+
13+
- [ ] I've confirmed the [contribution guidelines](https://github.com/ddmms/mlip-testing/blob/main/contributing.md).
14+
15+
## Summary
16+
17+
<!-- Describe your proposed changes. This can be brief, as most information can be in the linked issue. -->
18+
19+
## Linked issue
20+
21+
<!-- Enter the number of the issue this resolves. -->
22+
Resolves #
23+
24+
## Testing
25+
26+
<!-- How have your proposed changes been tested? -->
Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
<!--
2+
Thank you for contributing! To help us out with reviewing, please consider the following:
3+
4+
- Does this pull request include a summary of the change (see below)?
5+
- Does this pull request include a descriptive title?
6+
- Does this pull request link to a "new benchmark" issue (see below)?
7+
-->
8+
9+
## Pre-review checklist for PR author
10+
11+
PR author **must** check the checkboxes below when creating the PR.
12+
13+
- [ ] I've confirmed the [contribution guidelines](https://github.com/ddmms/mlip-testing/blob/main/contributing.md).
14+
15+
## Summary
16+
17+
<!-- Summarise the new benchmark. This can be brief, as most information should be in the linked issue. -->
18+
19+
## Linked issue
20+
21+
<!-- Enter the number of the issue this resolves. This should be labelled as "new benchmark" -->
22+
Resolves #
23+
24+
## Progress
25+
26+
<!-- Which aspects of the new benchmark are complete? -->
27+
- [ ] Calculations
28+
- [ ] Analysis
29+
- [ ] Application
30+
31+
## Testing
32+
33+
<!-- Which models have you tested your benchmark on? -->
34+
35+
## New decorators/callbacks
36+
37+
<!-- Are new callbacks required for the interactivity you need? -->
Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
<!--
2+
Thank you for contributing! To help us out with reviewing, please consider the following:
3+
4+
- Does this pull request include a summary of the change (see below)?
5+
- Does this pull request include a descriptive title?
6+
- Does this pull request link to a "new model" issue (see below)?
7+
-->
8+
9+
## Pre-review checklist for PR author
10+
11+
PR author **must** check the checkboxes below when creating the PR.
12+
13+
- [ ] I've confirmed the [contribution guidelines](https://github.com/ddmms/mlip-testing/blob/main/contributing.md).
14+
15+
## Summary
16+
17+
<!-- Describe the new model. This can be brief, as most information should be in the linked issue. -->
18+
19+
## Linked issue
20+
21+
<!-- Enter the number of the issue this resolves. This should be labelled as "new model" -->
22+
Resolves #
23+
24+
## Testing
25+
26+
<!-- Which benchmarks have you tested your model against? -->

0 commit comments

Comments
 (0)