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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add new OIL License #2423
Add new OIL License #2423
Conversation
Updating
Updating
Updating my fork
Updating my fork
Updating my fork
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.
Thanks!
I have a few comments for your consideration.
IMHO the correct text_url would be this:
- other_urls could have something such as
- homepage_url should be https://github.com/StarkDrones/OPNL
- The license key should be
opnl-1.0
andopnl-2.0
respectively as their authors name them asOPNL
notOIL
name: Open Innovation License version 1.0 | ||
category: Permissive | ||
owner: Stark Drones Corporation | ||
notes: From https://github.com/StarkDrones/OPNLv1/blob/main/.github/LICENSE.md |
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.
notes: From https://github.com/StarkDrones/OPNLv1/blob/main/.github/LICENSE.md | |
other_urls: | |
- https://github.com/StarkDrones/OPNLv1/blob/main/.github/LICENSE.md |
category: Permissive | ||
owner: Stark Drones Corporation | ||
notes: From https://github.com/StarkDrones/OPNLv1/blob/main/.github/LICENSE.md | ||
homepage_url: https://www.en.patent.aau.dk/open-innovation-licensing/ |
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.
I do not think this is the home page and that this Danish outfit is related to this license here. They vaguely use similar names. Or may be I am missing something?
category: Permissive | ||
owner: Stark Drones Corporation | ||
notes: From https://github.com/StarkDrones/OPNLv2/blob/main/.github/LICENSE.md | ||
homepage_url: https://www.en.patent.aau.dk/open-innovation-licensing/ |
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.
Same comment as above for notes and homepage
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.
Hey! @pombredanne I have made the requested changes . Please review !
Do not fail with empty lists
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.
@akugarg Can you rebase this into a single commit?
@akugarg LGTM! 馃憤 we just need a rebase before we can merge:) |
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
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.
LGTM! Thank you++
Merging!
Signed-off-by: Akanksha Garg akanksha.garg2k@gmail.com
Fixes #2384
Tasks
Run tests locally to check for errors.