license(vCluster): Updated OSS licenses#3865
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5544898657
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| - [k8s.io/utils](https://pkg.go.dev/k8s.io/utils) ([Apache-2.0](https://github.com/loft-sh/vcluster/blob/HEAD/vendor/k8s.io/utils/LICENSE)) | ||
| - [k8s.io/utils/internal/third_party/forked/golang](https://pkg.go.dev/k8s.io/utils/internal/third_party/forked/golang) ([BSD-3-Clause](https://github.com/loft-sh/vcluster/blob/HEAD/vendor/k8s.io/utils/internal/third_party/forked/golang/LICENSE)) | ||
| - [modernc.org/libc](https://pkg.go.dev/modernc.org/libc) ([BSD-3-Clause](https://github.com/loft-sh/vcluster/blob/HEAD/vendor/modernc.org/libc/LICENSE-GO)) | ||
| - [modernc.org/mathutil](https://pkg.go.dev/modernc.org/mathutil) ([Apache-2.0](Unknown)) |
There was a problem hiding this comment.
This license manifest entry is incorrect: it labels modernc.org/mathutil as Apache-2.0 and links to Unknown, but the vendored dependency includes an actual LICENSE file at vendor/modernc.org/mathutil/LICENSE with BSD-3-Clause terms. Because this page is explicitly for OSS attribution, publishing a wrong license and a broken license URL can cause compliance/audit errors for anyone relying on the generated list.
Useful? React with 👍 / 👎.
5544898 to
c9de92a
Compare
3434032 to
0a1d69d
Compare
Signed-off-by: Loft Bot <loft-bot@users.noreply.github.com>
0a1d69d to
6de3322
Compare
E2E Ginkgo Tests
|
Triggered by 5e86a09