Skip to content

Commit ae37766

Browse files
authored
Merge pull request #153 from aim-rsf/update-PR-template
Update pull_request_template.md
2 parents 65a4e43 + 3d85fe9 commit ae37766

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

.github/pull_request_template.md

+9-6
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,14 @@ References #
1515
-
1616
-
1717

18-
## Checklist before review:
18+
## Checklist for the author of this PR:
1919
<!-- You're invited to open a draft PR so people can see what you are working on sooner -->
20-
- [ ] Please comment on my PR while it's a draft and give me feedback on the development!
21-
- [ ] I added everything I wanted to add to this PR, please review!
22-
- [ ] [if package files were edited] Running `devtools::check()` returns no errors or warnings.
23-
- [ ] The code base and the documentation match (they both reflect any recent changes).
20+
- [ ] [if package files were edited] I have run these checks locally:
21+
- `devtools::document()` to generates the `.Rd` files from any updated roxygen comments.
22+
- `codemetar::write_codemeta()` to ensures the metadata file is up to date.
23+
- `styler::style_pkg()` to ensure consistent code styling that match the guidelines.
24+
- `devtools::check()` for a comprehensive package check. I have resolved any warnings or errors, or written them here in the PR, for discussion.
25+
- [ ] The code base and the documentation files match (they both reflect any recent changes).
2426
- [ ] The title of this PR is clear and self-explantory.
25-
- [ ] I added any appropriate labels to this PR.
27+
- [ ] I have added any appropriate labels to this PR.
28+
- [ ] This PR is now ready for review (and I have removed the draft PR status).

0 commit comments

Comments
 (0)