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

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

Merged
merged 894 commits into from
Aug 25, 2021

Conversation

sritasngh
Copy link
Contributor

@sritasngh sritasngh commented Jan 12, 2021

Signed-off-by: itssingh [email protected]
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 📁

Copy link
Member

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

src/licensedcode/data/licenses/anti-capitalist-1.0.LICENSE Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.0.yml Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.1.LICENSE Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.2.LICENSE Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.3.LICENSE Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.3.LICENSE Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.2.yml Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.2.LICENSE Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.1.yml Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.1.LICENSE Outdated Show resolved Hide resolved
@sritasngh sritasngh force-pushed the anti-capitalist branch 2 times, most recently from cae78e2 to f691361 Compare March 2, 2021 13:32
@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
Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

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

@sritasngh
Copy link
Contributor Author

sritasngh commented Mar 30, 2021

See the "ignorables"-related test failures

should I put url in notes: ?

Copy link
Member

@pombredanne pombredanne left a comment

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

src/licensedcode/data/licenses/anti-capitalist-1.4.yml Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.4.yml Outdated Show resolved Hide resolved
src/licensedcode/data/rules/anti_capitalist1.yml Outdated Show resolved Hide resolved
src/licensedcode/data/rules/anti_capitalist1.yml Outdated Show resolved Hide resolved
@sritasngh sritasngh requested a review from pombredanne June 3, 2021 10:57
Copy link
Member

@pombredanne pombredanne left a comment

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.

src/licensedcode/data/licenses/anti-capitalist-1.4.yml Outdated Show resolved Hide resolved
src/licensedcode/data/licenses/anti-capitalist-1.4.yml Outdated Show resolved Hide resolved
src/licensedcode/data/rules/anti_capitalist1.yml Outdated Show resolved Hide resolved
src/licensedcode/data/rules/anti_capitalist2.yml Outdated Show resolved Hide resolved
src/licensedcode/data/rules/anti_capitalist3.yml Outdated Show resolved Hide resolved
src/licensedcode/data/rules/anti_capitalist4.yml Outdated Show resolved Hide resolved
@sritasngh
Copy link
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 14 commits June 16, 2021 15:32
Signed-off-by: Philippe Ombredanne <[email protected]>
Update CONTRIBUTING.rst

Authored-by: Abhigya Verma<[email protected]>
Signed-off-by: Philippe Ombredanne <[email protected]>
Format code. Refactor common code between RDF and TV.

Signed-off-by: Philippe Ombredanne <[email protected]>
Signed-off-by: Philippe Ombredanne <[email protected]>
Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
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 <[email protected]>
…ting

Update Contributing: Development setup-instructions

Signed-off-by: Marcel Rettig <[email protected]>
Signed-off-by: Philippe Ombredanne <[email protected]>
@sritasngh
Copy link
Contributor Author

@pombredanne Should I add notes for all the rules?

akugarg and others added 22 commits August 10, 2021 23:29
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 <[email protected]>
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 <[email protected]>
Add emails and urls in converted_infos. Add holders, authors, emails and
urls information in html template.

Signed-off-by: Sarita Singh <[email protected]>
Co-authored-by: Avinal Kumar <[email protected]>
Signed-off-by: Sarita Singh <[email protected]>

Co-authored-by: Avinal Kumar <[email protected]>
Avoid misinterpreting MIT license notice as Apache-2.0, issue aboutcode-org#2635
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 <[email protected]>
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 <[email protected]>
This is no longer supported in newest Python versions.

Signed-off-by: Philippe Ombredanne <[email protected]>
@pombredanne
Copy link
Member

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

Copy link
Member

@pombredanne pombredanne left a comment

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