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

Editorial process updates #536

Merged
merged 13 commits into from Jun 7, 2019

Conversation

Projects
None yet
4 participants
@arfon
Copy link
Member

commented May 21, 2019

👋 @labarba @danielskatz @kyleniemeyer - I've tried to capture our discussion in this PR. Let me know what you think and if we're missing anything.

arfon added some commits May 21, 2019

- If the chosen editor has a lighter editorial load than other editors

Normally an EiC will ask one or more editors to edit a submission (e.g. `@editor1, @editor 2 - would one of you be willing to edit this submission for JOSS`). If the editor doesn't respond withing a 48 hours, the EiC may assign the paper to the editor regardless.

### Finding reviewers

At this point, the handling editor's job is to identify reviewer(s) who have sufficient expertise in the field of software and of the field of the submission. We like to have two reviewers per submission. If the editor is comfortable with their own assessment of the submission, one reviewer may be sufficient. If the editor feels particularly unsure of the submission, maybe a third reviewer can be recruited.

This comment has been minimized.

Copy link
@danielskatz

danielskatz May 21, 2019

Collaborator

here we say one review may be sufficient


**If reviews go stale**

Sometimes reviews go quiet, either because a reviewer has failed to complete their review or an author has been slow to respond to a reviewer's feedback. **It's your job as the editor to prompt the author/or reviewer(s) to revisit the submission if there has been no response within 7-10 days.**

This comment has been minimized.

Copy link
@danielskatz

danielskatz May 21, 2019

Collaborator

unless there's a clear statement in the review thread that says an action is coming at a slightly later time, perhaps because a reviewer committed to a review by a certain date, or an author is making changes and says they will be done by a certain date.

Show resolved Hide resolved docs/editing.md Outdated
Show resolved Hide resolved docs/editing.md Outdated

arfon added some commits May 21, 2019

@kyleniemeyer

This comment has been minimized.

Copy link
Collaborator

commented May 21, 2019

I'm good with the latest language. However, the expectation that all reviews have at minimum two reviewers will likely be news to some of our editors, who regularly have assigned one. (I've always tried to assign ≥2 reviewers myself.) So we'll just want to make sure and communicate that expectation, along with some of the other updates/clarifications.

@labarba

This comment has been minimized.

Copy link
Member

commented May 21, 2019

For a while now, I've been assigning a minimum of two reviewers, consistently.

We've seen various examples of reviewers going MIA mid-review, or never showing up after the Review issue is started. When two reviewers were assigned, we've been able to move forward to acceptance on the basis of the one responsive review, plus extra checks from the editor. When only the one delinquent reviewer was assigned, the editor needs to start from scratch finding a new reviewer, and it's a long delay for the paper.

@kyleniemeyer

This comment has been minimized.

Copy link
Collaborator

commented May 21, 2019

@labarba I totally agree, and I've been an advocate for at least two reviewers since the beginning—my only point is that in practice some editors have not been doing this consistently, so we need to make the expectation clear to all.

Show resolved Hide resolved docs/editing.md Outdated
Show resolved Hide resolved docs/editing.md Outdated
Show resolved Hide resolved docs/editing.md Outdated
Show resolved Hide resolved docs/editing.md Outdated
Show resolved Hide resolved docs/editing.md Outdated
Show resolved Hide resolved docs/editing.md Outdated

arfon added some commits Jun 4, 2019

@danielskatz

This comment has been minimized.

Copy link
Collaborator

commented Jun 7, 2019

Should we say something about the editor supplying a review in some cases? If so, this would be by formally adding themself as a reviewer, not just informally approving the paper? And would this be allowed for the 2nd review, or would there need to be 2 non-editor reviewers?

@arfon

This comment has been minimized.

Copy link
Member Author

commented Jun 7, 2019

👍

@arfon arfon merged commit 11df336 into master Jun 7, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@arfon arfon deleted the editorial-process-updates branch Jun 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.