-
-
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]: Web-based text anonymization with Node.js: Introducing NETANOS (Named entity-based Text Anonymization for Open Science) #293
Comments
Hello human, I'm @whedon. I'm here to help you with some common editorial tasks for JOSS. @RichardLitt 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:
|
Version: The GitHub release does not match, and the npm release is now ahead of the release for this review (as it is 1.1.3, instead of 1.1.0). I don't know how to best suggest moving forward; perhaps committing and tagging a GitHub release as 1.1.3, and then updating this submission here? |
Opened issues for Version and References. Otherwise, this is all good. Note: not sure what to do about versions. See above. |
Note: I am happy with the changes I suggested in #283. These have now been fixed. |
@RichardLitt Thanks for pointing that versioning issue out. We've tried to address this as follows:
Does this sound like an option? We would need to change the version for the JOSS submission then though. |
Sounds good to me! Changing the version in the JOSS submission also seems like a smart move. @arfon that's the last thing that needs to be done, and then I consider this to be worthy of accepting as is. |
@whedon commands |
Here are some things you can ask me to do:
🚧 Important 🚧 This is all quite new. Please make sure you check the top of the issue after running a @whedon command (you might also need to refresh the page to see the issue update). |
@arfon Looks like we need to have an 'Update version' command, and an 'accept paper' command. |
👍 yep, that's next on my todo list. |
@ben-aaron188 - Could you move the references you currently have in the I've also made a small fix to your paper meta data here: ben-aaron188/netanos#7 After fixing these two items up, 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. |
@arfon The references are updated and the pull request is merged. To avoid any versioning inaccuracies with npm, we’ve updated the npm release with these changes (which unfortunately implied that we needed to provide a new version in the We’ve linked the The bagde is here (already included at the top of the readme): |
Small note: thanks @ben-aaron188 for being so receptive to changes and fixes! |
@whedon set 10.5281/zenodo.817892 as archive |
OK. 10.5281/zenodo.817892 is the archive. |
@RichardLitt many thanks for your review here ✨ @ben-aaron188 - your paper is now accepted into JOSS and your DOI is http://dx.doi.org/10.21105/joss.00293 ⚡️:rocket: :boom: |
@RichardLitt Thanks for your helpful review and for taking the time with this tool! Much appreciated. @arfon What a fantastic, smooth review process this is. I will recommend your journal to colleagues eager to put their open source research software out there! I think this kind of open reviewing will be the future of academic peer reviewing soon! |
💖 Thanks! |
Submitting author: @ben-aaron188 (Bennett Kleinberg)
Repository: https://github.com/ben-aaron188/netanos
Version: v1.1.5
Editor: @acabunoc
Reviewer: @RichardLitt
Archive: 10.5281/zenodo.817892
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: