Skip to content
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

License.txt #8

Closed
brylie opened this issue Sep 19, 2016 · 7 comments
Closed

License.txt #8

brylie opened this issue Sep 19, 2016 · 7 comments

Comments

@brylie
Copy link

@brylie brylie commented Sep 19, 2016

I am reviewing this package as part of a submission to the Journal of Open Source Software.

As part of my review, I am instructed to locate a license file for the sourcecode. Where can I locate the LICENSE.text, or equivalent file for your wrapper package code?

@brylie brylie mentioned this issue Sep 19, 2016
15 of 16 tasks complete
@arfon
Copy link

@arfon arfon commented Sep 19, 2016

@brylie - R packages usually include this in the DESCRIPTION file (see here). While this isn't ideal, CRAN (the R package manager) won't permit plain text LICENSE files so what we currently have is compatible with JOSS requirements.

@carlganz
Copy link
Owner

@carlganz carlganz commented Sep 19, 2016

I can add a licence file to the github repo if you'd like.

Thanks for the review everyone!

@arfon
Copy link

@arfon arfon commented Sep 19, 2016

I can add a licence file to the github repo if you'd like.

Sounds good to me @carlganz. Thanks.

@carlganz
Copy link
Owner

@carlganz carlganz commented Sep 21, 2016

Added license

@graingert
Copy link

@graingert graingert commented Sep 22, 2016

FYI this should be the AGPL-3 because it uses/links against an AGPL-3 library

@carlganz
Copy link
Owner

@carlganz carlganz commented Sep 22, 2016

Thanks

@carlganz
Copy link
Owner

@carlganz carlganz commented Sep 27, 2016

@carlganz carlganz closed this Oct 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.