-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
[REVIEW]: Weave.jl: Scientific Reports Using Julia #204
Comments
Hello human, I'm @whedon. I'm here to help you with some common editorial tasks for JOSS. @vchuravy it looks like you're currently assigned as the reviewer for this paper 🎉. ⭐ Important ⭐ If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As as reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all JOSS reviews 😿 To fix this do the following two things:
For a list of things I can do to help you, just type:
|
@arfon How would I update the version number of the package for submission? I am nearly done with my review, but I think a point or minor release would be good to give a smooth user experience. |
Also didn't we used to generate a preview of the PDF? |
@vchuravy - I can do this. What's the new release number?
Yeah, I usually do this at the end of the review during processing now. We plan to automate this in the future. |
The new version is v0.4.1. |
@vchuravy - here's the compiled paper PDF for your review 10.21105.joss.00204.pdf |
Nice! @mpastell updated one citation and I think we are good to go here 👍 |
@arfon anything else left to do? |
@vchuravy - if this is all good then it's over to myself and @mpastell to wrap this up. @mpastell - At this point could you make an archive of the reviewed software in Zenodo/figshare/other service and update this thread with the DOI of the archive? I can then move forward with accepting the submission. |
@whedon set 10.5281/zenodo.398871 as archive |
OK. 10.5281/zenodo.398871 is the archive. |
@vchuravy many thanks for your rapid review here ✨ @mpastell - your paper is now accepted into JOSS and your DOI is http://dx.doi.org/10.21105/joss.00204 ⚡️ 🚀 💥 |
Submitting author: @mpastell (Matti Pastell)
Repository: https://github.com/mpastell/weave.jl
Version: v0.4.1
Editor: @arfon
Reviewer: @vchuravy
Archive: 10.5281/zenodo.398871
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer questions
Conflict of interest
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?The text was updated successfully, but these errors were encountered: