Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update pull_request_template.md #153

Merged
merged 2 commits into from
Nov 27, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 9 additions & 6 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,14 @@ References #
-
-

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