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

OSI approved license required by JOSS #1

Merged
merged 3 commits into from
May 10, 2021

Conversation

dlebauer
Copy link
Contributor

@dlebauer dlebauer commented Mar 18, 2021

You can choose one from here: https://opensource.org/licenses

This PR has BSD 3-clause which (like MIT and a few others) are 'permissive' - don't have restrictions like GPL family (which are also approved)

This now removes the LICENSE file that conflicts w/ LICENSE.md

result is that Github should correctly identify this as having an OSI approved license

You can choose one from here: https://opensource.org/licenses

This PR has BSD 3-clause which (like MIT and a few others) are 'permissive' - don't have restrictions like GPL family (which are also approved)
MIT info is in LICENSE.md
@DidierMurilloF
Copy link
Owner

Dear, @dlebauer thank you for the pull request. I am confused about the LICENCE.md file and why we should delete it.

Is not necessary that file? Thank you for your time.

Best,

Didier Murillo

@dlebauer
Copy link
Contributor Author

dlebauer commented May 7, 2021

Hi @DidierMurilloF

You have two license files with different content. LICENSE.md contains both the copyright and a copy of the the MIT License.

The LICENSE file only contains the copyright information (no license). Having two files is confusing.

This PR suggests deleting the LICENSE file that isn't actually a license.

And then renaming LICENSE.md --> LICENSE 3720ee0 because R CMD Check failed with message "Invalid license file pointers: LICENSE"

@DidierMurilloF DidierMurilloF merged commit ccb2f2c into DidierMurilloF:master May 10, 2021
andyquinterom pushed a commit to andyquinterom/FielDHub that referenced this pull request Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants