Skip to content

Initial ScanOSS Licensing Checker#2583

Open
arav-agarwal2 wants to merge 12 commits into
masterfrom
arav-patch-035
Open

Initial ScanOSS Licensing Checker#2583
arav-agarwal2 wants to merge 12 commits into
masterfrom
arav-patch-035

Conversation

@arav-agarwal2

Copy link
Copy Markdown
Contributor

This is a preliminary implementation of a scanoss-based licensing checker.

This does the following:

If a PR has more than 15 lines matched from a well-known repo, we force declaration of code used in scanoss. If undeclared, it will fail.
Currently this fails if a PR contains code that is found from a repo that is not copyleft. We only include BSD3, Apache2, and MIT currently.
Assisted By: claude opus-4.7

Mostly just to write the license check and figure out ranking thresholds.

( Redux from other so I can add ScanOSS key when necessary ).

@github-actions

github-actions Bot commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@arav-agarwal2 arav-agarwal2 marked this pull request as ready for review July 9, 2026 16:13
@arav-agarwal2 arav-agarwal2 requested review from a team as code owners July 9, 2026 16:13
@arav-agarwal2

Copy link
Copy Markdown
Contributor Author

Noting this before merging - I did get rate-limited after running 6 or so scans. I'll add some workaround logic and some strats to reduce rate exhaustion, but I'll think on other ways to manage this so that we get what we need out of this exchange. It's a bit annoying to get rate limited so quickly, but it is a free service.

@anandhu-eng anandhu-eng left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

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.

4 participants