Skip to content

dependency review: add 0BSD and Apache-2.0 variants#305

Merged
tigrato merged 1 commit intomainfrom
tigrato/add-licenses
Dec 20, 2024
Merged

dependency review: add 0BSD and Apache-2.0 variants#305
tigrato merged 1 commit intomainfrom
tigrato/add-licenses

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented Dec 20, 2024

Checks began failing this evening with the following issue:

The dependencies listed below have incompatible licenses:

  • pnpm-lock.yaml » prettier@3.4.2 – License: Apache-2.0 AND BSD-2-Clause AND BSD-3-Clause AND ISC AND MIT
  • pnpm-lock.yaml » rollup@4.28.1 – License: 0BSD AND ISC AND MIT

The rollup library itself is licensed under MIT, but some of its dependencies are under ISC and 0BSD licenses [1].

Similarly, prettier is released under the MIT license, but several of its dependencies use other licenses [2].

[1] https://github.com/rollup/rollup/blob/master/LICENSE.md
[2] https://github.com/prettier/prettier/blob/main/LICENSE

Checks started failing this evening with:

  The following dependencies have incompatible licenses:
  pnpm-lock.yaml » prettier@3.4.2 – License: Apache-2.0 AND BSD-2-Clause AND BSD-3-Clause AND ISC AND MIT
  pnpm-lock.yaml » rollup@4.28.1 – License: 0BSD AND ISC AND MIT

The `rollup` library is MIT but some of its dependencies are ISC and 0BSD [1]

The `prettier` is released under MIT but some of its dependencies use other licenses [2].

[1] https://github.com/rollup/rollup/blob/master/LICENSE.md
[2] https://github.com/prettier/prettier/blob/main/LICENSE

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
@tigrato tigrato requested a review from a team as a code owner December 20, 2024 16:41
@tigrato tigrato merged commit 2a22679 into main Dec 20, 2024
avatus added a commit that referenced this pull request Jan 22, 2025
Similar to #305
this PR will add another variant/combo and various licenses. Idealy we
could manage this in some sort of "check if each of these invidually"
are included, but idk how the legallity of licenses work so Ill just
follow prior work here

related PR failure
https://github.com/gravitational/teleport/actions/runs/12911052502/job/36002924223?pr=51356#step:3:19
@avatus avatus mentioned this pull request Jan 22, 2025
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