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

[INTERNAL] package.json: Add license check for dependencies #671

Merged
merged 1 commit into from
Dec 28, 2023

Conversation

RandomByte
Copy link
Member

@RandomByte RandomByte commented Dec 28, 2023

Only allow dependencies with licenses listed as Gold, Silver or Bronze at https://blueoakcouncil.org/list

Generally those should not cause us any trouble. Restricting this further to specific licenses currently in use by our dependencies might cause unnecessary maintenance efforts in the future due to the great variety of licenses being used.

Exceptions added for callsite (uses MIT) and yesno (uses BSD).

@coveralls
Copy link

coveralls commented Dec 28, 2023

Coverage Status

coverage: 96.63%. remained the same
when pulling b63b3cf on add-license-checks
into 792049d on main.

Only allow dependencies with licenses listed as Gold, Silver or Bronze
at https://blueoakcouncil.org/list

Generally those should not cause us any trouble. Restricting this
further to specific licenses currently in use by our dependencies might
cause unnecessary maintenance efforts in the future due to the great
variety of licenses being used.

Exceptions added for "callsite" (uses MIT) and "yesno" (uses BSD).
@RandomByte RandomByte merged commit 5a5128f into main Dec 28, 2023
17 checks passed
@RandomByte RandomByte deleted the add-license-checks branch December 28, 2023 11:16
flovogt added a commit to SAP/ui5-tooling-extensions that referenced this pull request Dec 29, 2023
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.

3 participants