-
Notifications
You must be signed in to change notification settings - Fork 33
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
JOSE Review Comments #292
Comments
Thanks for the comments and feedback @jhollist! We really appreciate them :) We'll work to address these changes! |
This was referenced May 27, 2019
Merged
@jhollist Sorry for the delay. We've updated the repo as per your requests! |
@lwjohnst86 Looks good! Heading over the finish up my checklist now. Congrats, this is a really nice looking course! |
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
All, very nice job on the course and the JOSE submission!
I am mostly finished with the review over at openjournals/jose-reviews#49 but had a few specific comments to leave here. Once these are addressed, I should be able to complete the review checklist.
As I mention on the JOSE issue, I though these course materials were excellent. Will certainly be recommending them.
Any question about these comments, just let me know.
Specific Comments
Authorship
Installation Instructions and Usage
Community Guidelines
References
The text was updated successfully, but these errors were encountered: