Skip to content

[WIP] Add license detection rule for Uthmanic - #3226

Closed
ninad365 wants to merge 12 commits into
aboutcode-org:developfrom
ninad365:develop
Closed

ninad365 wants to merge 12 commits into
aboutcode-org:developfrom
ninad365:develop

Conversation

@ninad365

@ninad365 ninad365 commented Jan 26, 2023

Copy link
Copy Markdown

Signed-off-by: Ninad Ambade ninadambade@gmail.com
Fixes #3225

Tasks

  • 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 📁

@ninad365 ninad365 changed the title Add license detection rule for Uthmanic WIP: Add license detection rule for Uthmanic Jan 26, 2023
@ninad365 ninad365 changed the title WIP: Add license detection rule for Uthmanic [WIP]: Add license detection rule for Uthmanic Jan 26, 2023
@ninad365 ninad365 changed the title [WIP]: Add license detection rule for Uthmanic [WIP] Add license detection rule for Uthmanic Jan 26, 2023

@AyanSinhaMahapatra AyanSinhaMahapatra 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.

  1. See https://dev.azure.com/nexB/scancode-toolkit/_build/results?buildId=9807&view=logs&j=5ccb487c-030c-5c4a-6db4-16909ae49dc7&t=b50f1525-6eb8-588a-b5ae-b2a59d2d4632, you also have to add a new .LICENSE file for this, for this to be added as a valid license-key.

  2. This has ~320 appearances in github: https://github.com/search?q=King+Fahd+Glorious+Quran+Printing+Complex&type=code, and we can probably add more rules from here?

@ninad365

Copy link
Copy Markdown
Author

Closing this pull request to avoid unnecessary clutter, creating a new one #3234

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.

Add new proprietary license detection rule

2 participants