Skip to content
Open
Show file tree
Hide file tree
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
1 change: 1 addition & 0 deletions booknews.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

## Dev version

- 2026-08-24, enforce the prefixing of comments more (#1024).
- 2026-08-24, mention `NEWS.md` file in packaging guide (#1027).
- 2026-08-24, fix lifecycle badge (`@joelnitta`, #1030).
- 2026-06-26, explicitely state that authors should not change their package while it is under review (#1022).
Expand Down
3 changes: 2 additions & 1 deletion softwarereview_author.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,8 @@ If you intend to submit an accompanying manuscript for your package, rOpenSci ha
- An [editor](#editors) will review your submission within 5 business days and respond with next steps. The editor may assign the package to reviewers, request that the package be updated to meet minimal criteria before review, or reject the package due to lack of fit or overlap.
- If your package meets minimal criteria, the editor will assign 1-3 reviewers. They will be asked to provide reviews as comments on your issue within 3 weeks.
- While your package is under review, please refrain from making any change, apart from hot fixes. Indeed, reviewers should be able to review the latest (best until now) version of your software, and that version should not be changing.
- We ask that you respond to reviewers' comments within 2 weeks of the last-submitted review, but you may make updates to your package or respond at any time. Your response should include a link to the updated [NEWS.md](#news) of your package. Here is [an author response example](https://github.com/ropensci/software-review/issues/160#issuecomment-355043656). Once the response is commited, [submit it using the bot](#submit-response-to-reviewers). We encourage ongoing conversations between authors and reviewers. See the [reviewing guide](#reviewerguide) for more details.
- We ask that you respond to reviewers' comments within 2 weeks of the last-submitted review, but you may make updates to your package or respond at any time. Your response should include a link to the updated [NEWS.md](#news) of your package. Here is [an author response example](https://github.com/ropensci/software-review/issues/160#issuecomment-355043656). Once the response is commited, [submit it using the bot](#submit-response-to-reviewers). If reviewers [used prefixes](#prefixing) before their comments e.g. ml01, ml02, use them in your response as well.
- We encourage ongoing conversations between authors and reviewers. See the [reviewing guide](#reviewerguide) for more details.
- Any time package changes are likely to alter the results of [the automated `pkgcheck` checks](https://docs.ropensci.org/pkgcheck), authors can request a re-check with the command, `@ropensci-review-bot check package`.
- Please notify us immediately if you are no longer able to maintain your package or to respond to reviews. You will then be expected to either retract a submission, or to find alternative package maintainers. You can also discuss maintenance issues in the rOpenSci slack workspace.
- Once your package is approved, we will provide further instructions about the transfer of your repository to the rOpenSci repository.
Expand Down
1 change: 1 addition & 0 deletions softwarereview_editor.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -231,6 +231,7 @@ Please use all of this information to judge whether:
#### Initial editorial comments

- After automatic checks are posted, use the [editor template](#editortemplate) to guide initial checks (if not already covered by the EiC) and record your response to the submission.
Prefix each of your comments with an [identifier consisting of your initials and a number](#prefixing) e.g. `ml01`, `ml02`, etc.
You can also streamline your editor checks by using the [`pkgreviewr` package created by former editor Anna Krystalli](https://docs.ropensci.org/pkgreviewr/articles/editors.html).
Please strive to finish the checks and start looking for reviewers within 5 working days.
- Check against policies for [fit](#aims-and-scope) and [overlap](#overlap).
Expand Down
37 changes: 35 additions & 2 deletions softwarereview_reviewer.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,9 @@ All submissions trigger a detailed report on package structure and functionality

To review a package, please begin by copying our [review template](#reviewtemplate), or the the [statistics review template](https://stats-devguide.ropensci.org/pkgreview.html#pkgrev-template) for statistical software.
These templates should be used as high-level checklists to guide initial review structure.

#### Checklist

In addition to checking off the minimum criteria, we ask that you provide general comments addressing the following:

- Does the package comply with the [rOpenSci packaging guide](#building)?
Expand All @@ -55,8 +58,9 @@ In addition to checking off the minimum criteria, we ask that you provide genera
- Were functions and arguments named to work together to form a common, logical programming API that is easy to read, and autocomplete?
- If you have your own relevant data/problem, work through it with the package. You may find rough edges and use-cases the author didn't think about.

Some items in our checklist were inspired by the [Mozilla code review guide](https://mozillascience.github.io/codeReview/review.html).

Please be respectful and kind to the authors in your reviews. Our [code of conduct](#code-of-conduct) is mandatory for everyone involved in our review process. We expect you to submit your review within 3 weeks, depending on the deadline set by the editor. Please contact the editor directly or in the submission thread to inform them about possible delays.
#### Automatic tools

We encourage you to use automated tools to facilitate your review. These include:

Expand All @@ -68,7 +72,36 @@ We encourage you to use automated tools to facilitate your review. These includ

Reviewers may also re-generate package check results from `@ropensci-review-bot` at any time by issuing the single comment in a review issue: `@ropensci-review-bot check package`.

Some items in our checklist were inspired by the [Mozilla code review guide](https://mozillascience.github.io/codeReview/review.html).
#### Tone

Please be respectful and kind to the authors in your reviews. Our [code of conduct](#code-of-conduct) is mandatory for everyone involved in our review process. We expect you to submit your review within 3 weeks, depending on the deadline set by the editor. Please contact the editor directly or in the submission thread to inform them about possible delays.


#### Formatting {#prefixing}

You might be writing many individual comments as part of your review.
To facilitate communication around them, please prefix your answers like so, with your initials unless you share them with anyone else in the thread, and a number:

```
- ml01: your package lacks a test for bla.
- ml02: the reference index's grouping is unclear.
....
```

This way the author can respond like so:

```
> - ml01: your package lacks a test for bla.

Good catch, I added two tests, see [commit](link-to-commit).

> - ml02: the reference index's grouping is unclear.

Thank you, the functions are now grouped into data ingestion, data checking and data analysis. See [new reference index](pkgdown-link).

....
```


### Off-thread interactions {#off-thread-interactions}

Expand Down
2 changes: 2 additions & 0 deletions templates/editor.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,5 +14,7 @@

#### Editor comments

_Prefix all your comments with numbered codes e.g. ml01, ml02, etc._

---

2 changes: 2 additions & 0 deletions templates/review.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,3 +40,5 @@ Estimated hours spent reviewing:
---

### Review Comments

_Prefix all your comments with numbered codes e.g. ml01, ml02, etc._