Skip to content

Commit 9ae3169

Browse files
authored
Update PULL_REQUEST_TEMPLATE.md
1 parent 7773041 commit 9ae3169

1 file changed

Lines changed: 50 additions & 16 deletions

File tree

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 50 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,35 +1,69 @@
1-
**Description**
2-
1+
## Checklist before requesting a review
2+
<!-- partly taken from https://axolo.co/blog/p/part-3-github-pull-request-template -->
33
- [ ] I have read the [contribution guidelines](https://github.com/NLeSC/python-template/blob/main/CONTRIBUTING.md)
4-
- [ ] This update is in line with what is recommended in the [Python chapter of the Guide](https://guide.esciencecenter.nl/#/best_practices/language_guides/python)
4+
- [ ] My code follows the style guidelines of this project
5+
- [ ] I have performed a self-review of my code
6+
- [ ] I have commented my code, particularly in hard-to-understand areas
7+
- [ ] I have made corresponding changes to the documentation
8+
- [ ] My changes generate no new warnings
9+
- [ ] I have added tests that prove my fix is effective or that my feature works
10+
- [ ] New and existing unit tests pass locally with my changes
511
- [ ] All user facing changes have been added to CHANGELOG.md
12+
<!-- - [ ] Any dependent changes have been merged and published in downstream modules -->
13+
14+
## Type of change
15+
16+
Please delete options that are not relevant.
17+
18+
- [ ] Bug fix (non-breaking change which fixes an issue)
19+
- [ ] New feature (non-breaking change which adds functionality)
20+
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
21+
- [ ] This change requires a documentation update
22+
23+
## List of related issues or pull requests**
624

7-
<!-- Description of PR -->
25+
<!-- add all related issues to the list below -->
26+
Refs:
27+
- #ISSUE_NUMBER_1
28+
- #ISSUE_NUMBER_2
829

9-
<!--
10-
**Related issues**:
11-
- ...
12-
-->
30+
## Describe the changes made in this pull request
31+
32+
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
1333

1434
**Instructions to review the pull request**
1535

16-
<!-- remove what doesn't apply or add more if needed -->
17-
Create a `python-template-test` repo on GitHub (will be overwritten if existing)
36+
<!-- remove/update what doesn't apply or add more if needed -->
37+
38+
Install the requirements
39+
1840
```
19-
# Create a temporary directory by running the following command. Keep the XXXXXX in the directory name.
2041
cd $(mktemp -d --tmpdir py-tmpl-XXXXXX)
21-
# Use --vcs-ref <pr-branch> to point to the branch you want to test
22-
copier copy --vcs-ref <pr-branch> https://github.com/<pr-user>/python-template .
23-
# Fill with python-template-test info
24-
# Create a local git repo to push to GitHub to trigger CI actions
42+
pip install pipx
43+
pipx install copier
44+
```
45+
46+
Create a new package using the template
47+
48+
```
49+
copier copy --vcs-ref <YOUR_BRANCH> https://github.com/nlesc/python-template test_package
50+
```
51+
52+
Create a local git repo to push to GitHub to trigger CI actions
53+
```
2554
git init
2655
git add --all
2756
git commit -m "First commit"
28-
git remote add origin git@github.com:<you>/python-template-test.git
57+
git remote add origin git@github.com:<YOU>/python-template-test.git
2958
git push -u origin main -f
59+
```
60+
61+
```
3062
# Create a local environment to test your generated package locally
3163
python -m venv env
3264
source env/bin/activate
3365
python -m pip install --upgrade pip setuptools
3466
python -m pip install '.[dev,publishing]'
3567
```
68+
69+
<!-- Other steps -->

0 commit comments

Comments
 (0)