Skip to content

fix(ci): allow Python-2.0/0BSD in root license check - #1354

Open
saidai-bhuvanesh wants to merge 1 commit into
Nitya-003:mainfrom
saidai-bhuvanesh:fix/license-check-allow-python2
Open

fix(ci): allow Python-2.0/0BSD in root license check#1354
saidai-bhuvanesh wants to merge 1 commit into
Nitya-003:mainfrom
saidai-bhuvanesh:fix/license-check-allow-python2

Conversation

@saidai-bhuvanesh

@saidai-bhuvanesh saidai-bhuvanesh commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Aligns the Check Root Licenses step's --onlyAllow list with its sibling steps and adds the permissive BlueOak-1.0.0.

This PR was created by an AI agent (OpenHands) on behalf of saidai-bhuvanesh.

What this fixes (a config inconsistency)

In .github/workflows/license-check.yml, the Check Root Licenses step was missing Python-2.0 and 0BSD even though the Frontend/Backend/Smart-Contracts steps already allowed them. The root tree contains argparse@2.0.1 (Python-2.0) and minimatch@10.2.6 (BlueOak-1.0.0), so the root check failed on every PR. This PR adds Python-2.0;0BSD;BlueOak-1.0.0 to the root step and BlueOak-1.0.0 to the other steps. All three are permissive, SPDX-recognized licenses.

⚠️ Separate issue requiring a MAINTAINER decision (not addressed here)

Even after this fix, the Frontend license check still fails:

Package "@img/sharp-libvips-linux-x64@1.2.4" is licensed under "LGPL-3.0-or-later" which is not permitted by the --onlyAllow flag. Exiting.

@img/sharp-libvips-* is a transitive dependency of sharp. LGPL-3.0-or-later is a copyleft license with source-availability obligations, so allowing it is a project licensing-policy decision only the repo owner can make — I have not added it. Options for the owner:

  1. Add LGPL-3.0-or-later to the --onlyAllow lists (accepts the LGPL obligations), or
  2. Remove/replace the sharp dependency, or
  3. Document the LGPL compliance approach.

Note

Other CropChain CI failures (frontend build breakage from duplicated .catch chains / sourceIndex already declared) are tracked separately in #1227.

@vercel

vercel Bot commented Aug 15, 2026

Copy link
Copy Markdown

@openhands-agent is attempting to deploy a commit to the Nitya Gosain's projects Team on Vercel.

A member of the Team first needs to authorize it.

The 'Check Root Licenses' step's --onlyAllow list was missing Python-2.0
and 0BSD (already present in the frontend/backend/smart-contracts steps),
and none of the steps allowed BlueOak-1.0.0. The root dependency tree
contains argparse@2.0.1 (Python-2.0) and minimatch@10.2.6 (BlueOak-1.0.0),
so the license check failed on every PR.

Add Python-2.0, 0BSD and BlueOak-1.0.0 (all permissive/SPDX-recognized)
to the root step, and add BlueOak-1.0.0 to the other three steps for
consistency.

Co-authored-by: openhands <openhands@all-hands.dev>
@saidai-bhuvanesh
saidai-bhuvanesh force-pushed the fix/license-check-allow-python2 branch from d58b5c0 to 752edb0 Compare August 15, 2026 06:06
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.

2 participants