Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

chore(licenses): fix khroma and elkjs#62762

Merged
jhchabran merged 1 commit into
mainfrom
jh/update-licenses
May 17, 2024
Merged

chore(licenses): fix khroma and elkjs#62762
jhchabran merged 1 commit into
mainfrom
jh/update-licenses

Conversation

@jhchabran

Copy link
Copy Markdown
Contributor

Khroma and Elkjs were raising errors in the license checker:

  • ElkJS because its license is EPL-2.0 and we've never had that case before.
    • Added to allowed licenses.
  • Khroma is MIT but the NPM package is not up to date with regards to that info, and thus the license checker returns "unknown" when querying NPM.
    • Manually assigned the license.

Test plan

CI, in particular the license check action

@cla-bot cla-bot Bot added the cla-signed label May 17, 2024
@jhchabran jhchabran requested a review from a team May 17, 2024 13:32
@jhchabran jhchabran merged commit b024b8c into main May 17, 2024
@jhchabran jhchabran deleted the jh/update-licenses branch May 17, 2024 13:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants