-
Notifications
You must be signed in to change notification settings - Fork 33
enh: add new peer review lead role to the how to guides #348
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
Conversation
how-to/peer-review-lead.md
Outdated
| - Updating [software peer review policies](https://www.pyopensci.org/software-peer-review/our-process/policies.html) as needed | ||
| - Helping editors find reviewers as necessary. | ||
|
|
||
| This role will also help manage conflict if it arises in the peer review process. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think linking the dashboard is a good addition!
eliotwrobson
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the language sounds good! Still looks like in progress but the responsibilities seem reasonable, just managing the editorial staff and maintaining process documents.
how-to/peer-review-lead.md
Outdated
| - Updating [software peer review policies](https://www.pyopensci.org/software-peer-review/our-process/policies.html) as needed | ||
| - Helping editors find reviewers as necessary. | ||
|
|
||
| This role will also help manage conflict if it arises in the peer review process. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think linking the dashboard is a good addition!
|
ok @eliotwrobson, my apologies here - I didn't realize I hadn't finished this document. I've fleshed it out a bit more with the dashboard. I'll add some screenshots later today and will send you a rendered version when that's done. |
|
ok @eliotwrobson , here is an updated version I think we should work together as well to ensure this role isn't more hours than I think it will be. So we can always adjust things on the fly. I'll share this in Slack as well. This is failing ci because it's a new page and running against the live book. but everything is good to merge if we are both still happy with this as is. |
Co-authored-by: Leah Wasser <[email protected]>
Co-authored-by: Leah Wasser <[email protected]>
eliotwrobson
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lwasser went through the text and everything looks good! We can always revise the list of responsibilities as needed later on. I think this is a comprehensive starting point.
|
fantastic. Ok i'll clean things up here and will merge!! |
|
@eliotwrobson thank you for working on this with me!! 🚀 |
|
@all-contributors please add @eliotwrobson for docs, review |
|
I've put up a pull request to add @eliotwrobson! 🎉 |
No description provided.