-
-
Notifications
You must be signed in to change notification settings - Fork 582
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 new licenses #2625
Add new licenses #2625
Conversation
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
bff345a
to
fe08a5f
Compare
22b416d
to
19da689
Compare
@pombredanne the failing check mentioned here ,is it related to something that I have added? |
key: dpl | ||
short_name: DPL | ||
name: THE DEFENSIVE PATENT LICENSE v1.1 | ||
owner: Unspecified |
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.
It must have an owner
19da689
to
11f9e6e
Compare
Yes and no... I think we could remove entirely that test failing tests (tests/licensedcode/test_query.py::TestQueryWithFullIndex::test_Query_tokens_by_line_behaves_the_same_on_various_python ) and its data files as this only served a purpose when we were porting from Python 2 to Python 3. Nowadays it only ever fails when we have some significant new words added in new licenses... and that's useless since we are all on Python 3 |
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
c69214f
to
50b4fcc
Compare
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
0fc34a0
to
d954e40
Compare
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! Thanks and merging.
This PR new licenses
Tasks
Run tests locally to check for errors.