-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add follow-up to data licensing post #360
Conversation
✅ Deploy Preview for tourmaline-marshmallow-241b40 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@chartgerink once you've had time to review and compare both options please let me know what you prefer and feel free to suggest changes. |
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.
This is looking good. It makes more sense to me this way, as the chronology is clearer and there is less reading between the lines necessary.
I left a few suggestions, and my only question is whether I should even be an author on this, because you did virtually all of the work!
Co-authored-by: Chris Hartgerink <chris@libscie.org>
Co-authored-by: Chris Hartgerink <chris@libscie.org>
Co-authored-by: Chris Hartgerink <chris@libscie.org>
Thanks for the suggestions @chartgerink. Given your involvement in discussing and formulating the structure of the post and what to include, your contribution was still substantial and warrants authorship in my opinion. I'm happy for this to be merged. |
✅ Merge Schedule |
This PR adds a follow-up post to the Dual licensing R packages with code and data blog post authored by @chartgerink and myself.
This post provides a short summary of the original post and then provides an update to the Licensing code and data in one R package section including our experience since writing the blog post developing {epiparameter} and getting it hosted on CRAN.
I've added a callout to the the first version of the blog post in the Licensing code and data in one R package section to direct readers to updated blog post.
This PR provides an alternative option to PR #359. The preferred option can be merged and the other PR can be closed without merging.
R
Right before merging:
date
field has been updatedblueprints
to link to this post_freeze/
folder is up-to-date/schedule 2025-01-27T08:00