Skip to content

Add new license Anti-Capitalist Software License #2362 - #2364

Merged
pombredanne merged 894 commits into
aboutcode-org:developfrom
sritasngh:anti-capitalist
Aug 25, 2021
Merged

Add new license Anti-Capitalist Software License #2362#2364
pombredanne merged 894 commits into
aboutcode-org:developfrom
sritasngh:anti-capitalist

Conversation

@sritasngh

@sritasngh sritasngh commented Jan 12, 2021

Copy link
Copy Markdown
Contributor

Signed-off-by: itssingh saritasingh.0425@gmail.com
Fixes #2362

Added new licenses ANTI-CAPITALIST SOFTWARE LICENSE v-1.0, v-1.1, v-1.2, v-1.3 and v-1.4

These added licenses are giving correct scan result.

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

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 nitpickings for your consideration before we can merge.

Comment thread src/licensedcode/data/licenses/anti-capitalist-1.0.LICENSE Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.0.yml Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.1.LICENSE Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.2.LICENSE Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.3.LICENSE Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.3.LICENSE Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.2.yml Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.2.LICENSE Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.1.yml Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.1.LICENSE Outdated
@sritasngh
sritasngh requested a review from pombredanne March 13, 2021 14:44
@pombredanne pombredanne changed the title Added new license Anti-Capitalist Software License #2362 Add new license Anti-Capitalist Software License #2362 Mar 29, 2021

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sritasngh

sritasngh commented Mar 30, 2021

Copy link
Copy Markdown
Contributor Author

See the "ignorables"-related test failures

should I put url in notes: ?

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should I put url in notes:

no, you just need to add ignorable_* fields for URLs and emails

Comment thread src/licensedcode/data/licenses/anti-capitalist-1.4.yml Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.4.yml Outdated
Comment thread src/licensedcode/data/rules/anti_capitalist1.yml Outdated
Comment thread src/licensedcode/data/rules/anti_capitalist1.yml Outdated
@sritasngh
sritasngh requested a review from pombredanne June 3, 2021 10:57

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Please consider the few cleanups I suggested. Also feel free to squash all your commits in a single one.

Comment thread src/licensedcode/data/licenses/anti-capitalist-1.4.yml Outdated
Comment thread src/licensedcode/data/licenses/anti-capitalist-1.4.yml Outdated
Comment thread src/licensedcode/data/rules/anti_capitalist1.yml Outdated
Comment thread src/licensedcode/data/rules/anti_capitalist2.yml Outdated
Comment thread src/licensedcode/data/rules/anti_capitalist3.yml Outdated
Comment thread src/licensedcode/data/rules/anti_capitalist4.yml Outdated
@sritasngh

Copy link
Copy Markdown
Contributor Author

consider the few cleanups I suggested

I left these #2364 (comment) as trailing line return #2364 (comment) as you previously suggested. Removed now :)

@sritasngh
sritasngh requested a review from pombredanne June 4, 2021 01:26
pombredanne and others added 20 commits June 16, 2021 15:32
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Update CONTRIBUTING.rst

Authored-by: Abhigya Verma<abhigyaverma27@gmail.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Format code. Refactor common code between RDF and TV.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
pombredanne and others added 3 commits August 6, 2021 15:37
Add new flag in License Data Model definition
Some instructions were not (exactly) right, and some commands are now also highlighted properly.

Signed-off-by: Marcel Rettig <mapelpapel@gmail.com>
Update Contributing: Development setup-instructions

Signed-off-by: Marcel Rettig <mapelpapel@gmail.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@sritasngh

Copy link
Copy Markdown
Contributor Author

@pombredanne Should I add notes for all the rules?

akugarg and others added 22 commits August 10, 2021 23:29
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
Referenced_filenames should be returned by API function
This provides a rule that matches MIT notices of form (note: indent for highlight):

  License

  This project is licensed under the terms of the MIT license.

Signed-off-by: Peter Gardfjäll <peter.gardfjall.work@gmail.com>
This avoids Apache-2.0 "overriding" mit_349.RULE in cases like

  ## License

  This project is licensed under the terms of the MIT license.

Signed-off-by: Peter Gardfjäll <peter.gardfjall.work@gmail.com>
Add emails and urls in converted_infos. Add holders, authors, emails and
urls information in html template.

Signed-off-by: Sarita Singh <saritasingh.0425@gmail.com>
Co-authored-by: Avinal Kumar <avinal.xlvii@gmail.com>
Signed-off-by: Sarita Singh <saritasingh.0425@gmail.com>

Co-authored-by: Avinal Kumar <avinal.xlvii@gmail.com>
Add emails and urls to HTML output
Avoid misinterpreting MIT license notice as Apache-2.0, issue #2635
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
Signed-off-by: akugarg <akanksha.garg2k@gmail.com>
This version check causes problems when running in a firewalled environment (in my case, hanging for 30 seconds before timing out). This just adds a CLI option to skip it entirely.

Signed-off-by: Yunus Rahbar <yrahbar@fb.com>
Add "--no-check-version" CLI option to scancode
Align tests for pubspec with latest code
Somehow a tab had squeezed in

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
This is no longer supported in newest Python versions.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@pombredanne

Copy link
Copy Markdown
Member

FYI I merged the latest develop and fixed the tests, then pushed back to your branch

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Merging now at last. Thanks!

@pombredanne
pombredanne merged commit 3a82423 into aboutcode-org:develop Aug 25, 2021
@sritasngh
sritasngh deleted the anti-capitalist branch October 9, 2021 12:50
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.

Incorrect License detected for Anti-Capitalist Software License