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]: Containershare: Open Source Registry to build, test, deploy with CircleCI #878
Comments
Hello human, I'm @whedon. I'm here to help you with some common editorial tasks. @brainstorm 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 a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿 To fix this do the following two things:
For a list of things I can do to help you, just type:
|
|
Alright, let's start! Thanks @arfon 👍 @vsoch Your project/repo seems quite well put together, but I have one question you'll have to motivate a bit more on In other words, |
Could you also have a small mention on how to run the unit tests? ;) |
Thanks for the speedy update @brainstorm and @arfon (and @whedon!) I've done the following changes to address the review comments:
I think that should be good for this first tweak - take a look and let me know what needs more work! I'll also note that I have several new container templates in the queue - the exciting thing about this work is that it isn't a static "it's done now and forever" thing, it's going to be extended to have lots of different container types (templates) that a user can add to the containershare! |
I just edited @whedon 's version up to what we will tag as the final release, after review: **Version:** 0.0.3 I didn't realize I could do that :) |
heyo everyone! For a full circle test of the above changes, I added a repo2docker-julia template just now, and everything worked as I would want and expect! You can see: And upon merge, the table and associated linked metadata was updated. |
Cool, thanks for the quick changes @vsoch, all LGTM now. I would suggest to add a bib reference to biocontainers since you mention them... not sure they even have a preprint though, care to contact and ask them?: http://biocontainers.pro/#contact Other than that, all clear, good job! ;) |
oh hmm, I definitely did that! I think Mr.Whedon just needs to rebuild the pdf? Let me see if I can figure that out... @whedon commands |
@whedon commands |
Here are some things you can ask me to do:
|
@whedon generate pdf |
|
See it now? Thanks @whedon ! You da bomb' bot. |
Haha, great then, all clear from my side, @arfon, over to you ;) |
Thanks @brainstorm! You da bomb'man. :) |
@vsoch - 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. |
All set! version 0.0.3 with the final work is here --> https://zenodo.org/account/settings/github/repository/vsoch/containershare# pointing to https://github.com/vsoch/containershare/releases/tag/0.0.3. The link for the doi provided is https://zenodo.org/badge/latestdoi/142066803 (although not found yet?) Let me know if you need anything else! |
@whedon set 10.5281/zenodo.1341099 as archive |
OK. 10.5281/zenodo.1341099 is the archive. |
@brainstorm - many thanks for your review here ✨ @vsoch - your paper is now accepted into JOSS and your DOI is https://doi.org/10.21105/joss.00878 ⚡ 🚀 💥 |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
|
Woohoo! So awesome! Thank you @brainstorm @arfon, and of course, the fearless @whedon :) |
Submitting author: @vsoch (Vanessa Sochat)
Repository: https://www.github.com/vsoch/containershare
Version: 0.0.3
Editor: @arfon
Reviewer: @brainstorm
Archive: 10.5281/zenodo.1341099
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 instructions & questions
@brainstorm, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @arfon know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @brainstorm
Conflict of interest
Code of Conduct
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: