docs: add the missing Apache-2.0 LICENSE file - #358
Merged
Conversation
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe repository adds the complete Apache License, Version 2.0 text in a new ChangesRepository licensing
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Andrei Kvapil (kvaps)
marked this pull request as ready for review
August 12, 2026 20:57
Andrei Kvapil (kvaps)
requested review from
Andrey Kolkov (androndo) and
Timofei Larkin (lllamnyp)
as code owners
August 12, 2026 20:57
Andrey Kolkov (androndo)
approved these changes
Aug 13, 2026
Every source file in this repository carries the Apache-2.0 header and the README states "Apache 2.0. See LICENSE", but that file has never existed. GitHub therefore reports no licence for the repository, and the terms the sources refer to are not actually present. This adds the verbatim Apache License 2.0 text. No terms change: it makes explicit the licence the file headers have asserted since 2023. Assisted-By: Claude Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
Andrey Kolkov (androndo)
force-pushed
the
docs/add-license
branch
from
August 13, 2026 04:51
aef356b to
38ea862
Compare
Andrey Kolkov (androndo)
requested a review
from Kirill Ilin (sircthulhu)
as a code owner
August 13, 2026 04:51
Andrey Kolkov (androndo)
enabled auto-merge
August 13, 2026 04:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does
This repository has no
LICENSEfile. Not at the root, not anywhere in the tree —GET /repos/cozystack/etcd-operator/licensereturns 404 and GitHub reports no licence for the repository.Meanwhile every source file carries the Apache-2.0 header (
Copyright 2023 Timofey Larkin., seehack/boilerplate.go.txt) and the README states:So the terms the sources point at are not actually present in the repository.
This matters beyond tidiness:
etcd-operatorships into the Cozystack distribution three ways — the controller image (packages/system/etcd-operator/values.yaml), the CRDs (packages/system/etcd-operator-crds/Makefile), and theetcd-migratebinary baked into theplatform-migrationsimage (packages/core/platform/images/migrations/Dockerfile). A component in the artifact adopters install, with no repository-level licence grant, is a finding any licence review will raise.This adds the verbatim Apache License 2.0 text. No terms change: it makes explicit the licence the file headers have asserted since 2023.
Summary by CodeRabbit