Skip to content

Commit 12220a9

Browse files
authored
Create main_rev_checklist.md
1 parent c479c8b commit 12220a9

File tree

1 file changed

+27
-0
lines changed

1 file changed

+27
-0
lines changed
Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
Hello @__MAINREV__!
2+
3+
You've been assigned to this PR, which means that you've been nominated Main Reviewer! (Lucky you! :ta-da: :ta-da: :ta-da:)
4+
5+
As a Main Reviewer, you are slightly more responsible for the quality of this PR than your fellow Reviewers. If you have any doubt, check [this section on reviewing](https://phys2bids.readthedocs.io/en/latest/contributorfile.html#reviewing) and [this section on being the Main Reviewer](https://phys2bids.readthedocs.io/en/latest/contributorfile.html#mainreviewer) of the documents.
6+
7+
## Before merging this PR, please check that:
8+
- [ ] The PR passes all the CircleCI/Azure tests.
9+
- [ ] The content respects the [Style Guide](https://phys2bids.readthedocs.io/en/latest/contributorfile.html#styling).
10+
- [ ] If the PR contains documentation, your local build is right.
11+
- [ ] The PR and the reviews are harmonious with the rest of the repository, especially that **they don't introduce repetitions**
12+
- [ ] The title of the PR is clear enough to describe a release content.
13+
- [ ] The labels are correct - and they will (or won't) trigger the correct `auto` release.
14+
- [ ] The PR received the amount of approvals necessaries to be merged.
15+
16+
If the PR contains code changes:
17+
- [ ] Content and changes are adequately documented in the docstrings.
18+
- [ ] User documentation is being updated accordingly - or a related issue has been opened.
19+
- [ ] The adequate tests have been added/updated - or a related issue has been opened *if coverage doesn't drop below 90%*.
20+
21+
22+
## After you merged this PR, please check that:
23+
- [ ] The Author(s) and Reviewers contributions have been updated in the README.
24+
- [ ] The updates (in code, tests and documentation) have appened correctly.
25+
- [ ] If they had to, the tag was created, the release was cut, and the pypi version got updated.
26+
27+
Thank you!

0 commit comments

Comments
 (0)