docs: use verbatim Apache-2.0 license text#634
Open
dan2k3k4 wants to merge 2 commits into
Open
Conversation
The LICENSE file contained a paraphrased and truncated copy of Apache-2.0 (1247 words vs 1581 canonical, missing the APPENDIX and clauses in §8/§9). GitHub's licensee matches by fuzzy hash, so the repo was reported as NOASSERTION/"Other" rather than Apache-2.0. Replaced with the canonical text verbatim.
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.
Problem
LICENSEcontained a paraphrased and truncated copy of the Apache License 2.0 rather than the canonical text:APPENDIX: How to apply the Apache License to your worksection was missingGitHub's license detection (
licensee) matches by fuzzy hash, so it could not identify the license and reported the repo asNOASSERTION/ "Other":That blocks anything keyed off an OSI-approved license being detected — including our free Greptile reviews.
Change
Replaced
LICENSEwith the canonical Apache-2.0 text verbatim (gh api /licenses/apache-2.0). No change to the licensing intent — the file already said Apache-2.0, it just wasn't a faithful copy.The
APPENDIXcopyright line is left as the standardCopyright [yyyy] [name of copyright owner]placeholder, which is how most Apache-2.0 repos ship it and does not affect detection. Happy to fill in "amazee.io" if preferred.Verifying
Detection only re-runs against the default branch, so
gh api repos/amazeeio/amazee.ai --jq .licenseshould reportapache-2.0once this merges todev.Greptile Summary
This PR replaces a paraphrased, truncated copy of the Apache License 2.0 with the canonical verbatim text sourced from
gh api /licenses/apache-2.0, fixing GitHub'slicenseedetection which previously reported the repo asNOASSERTION/ "Other". There are no changes to code, configuration, or licensing intent.APPENDIX: How to apply the Apache License to your worksection, missing definition language in §1 (the"submitted"definition for"Contribution"), and missing clause fragments in §§5, 8, and 9.APPENDIXcopyright placeholder is left asCopyright [yyyy] [name of copyright owner], which is the standard convention and does not affect license detection.Confidence Score: 5/5
Safe to merge — this is a pure text replacement of the LICENSE file with no code changes whatsoever.
The change restores the verbatim Apache 2.0 text to fix GitHub license detection. No code, configuration, or logic is touched. The new text matches the canonical Apache-2.0 source and adds back the clauses and APPENDIX that were previously missing.
No files require special attention. The only changed file is LICENSE, and the replacement text is the well-known canonical Apache 2.0 license.
Important Files Changed
Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A[Old LICENSE\nParaphrased / truncated\n1247 words] -->|Replace with canonical text| B[New LICENSE\nVerbatim Apache-2.0\n1581 words] B --> C{GitHub licensee\nfuzzy-hash detection} C -->|Before PR| D[NOASSERTION / Other] C -->|After PR merges to dev| E[apache-2.0 ✓] E --> F[OSI-approved license detected\nEnables tools keyed on license]Reviews (1): Last reviewed commit: "docs: use verbatim Apache-2.0 license te..." | Re-trigger Greptile