Skip to content

License this repo as Apache 2.0#142

Merged
grahamc merged 1 commit intomainfrom
grahamc-patch-1
Dec 18, 2025
Merged

License this repo as Apache 2.0#142
grahamc merged 1 commit intomainfrom
grahamc-patch-1

Conversation

@grahamc
Copy link
Member

@grahamc grahamc commented Dec 18, 2025

Add Apache License 2.0 to the project.

Closes #141

Summary by CodeRabbit

  • Chores
    • Added Apache License 2.0 to the repository.

✏️ Tip: You can customize this high-level summary in your review settings.

Add Apache License 2.0 to the project
@grahamc grahamc mentioned this pull request Dec 18, 2025
@grahamc grahamc enabled auto-merge December 18, 2025 14:18
@coderabbitai
Copy link

coderabbitai bot commented Dec 18, 2025

Walkthrough

The pull request adds an Apache License, Version 2.0 document to the repository. The license file contains the complete legal text including definitions, grant terms, redistribution conditions, and disclaimer sections. No functional code or logic changes are introduced.

Changes

Cohort / File(s) Summary
License Documentation
LICENSE
Added complete Apache License, Version 2.0 text with standard definitions, grant sections, redistribution rules, contribution terms, trademark disclaimer, warranty limitations, and appendix instructions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Verify license text matches the official Apache License 2.0 template
  • Confirm file placement and naming convention are correct

Poem

🐰 A license now guards this burrow with care,
Apache's seal makes it open and fair,
With terms all declared in legal array,
Our work is protected—hip-hip-hooray! 🎉

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'License this repo as Apache 2.0' clearly and accurately describes the main change in the pull request, which adds an Apache License 2.0 file to the repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch grahamc-patch-1

Comment @coderabbitai help to get the list of available commands and usage tips.

@grahamc grahamc added this pull request to the merge queue Dec 18, 2025
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
LICENSE (1)

178-201: Consider adding copyright headers to source files.

The LICENSE file provides the legal framework, but Apache 2.0 licensing best practice recommends that each source file include a copyright notice and license header. This ensures clear provenance and licensing at the file level, particularly important for derivative works and distribution.

Verify that:

  • Source files include proper copyright notices (e.g., Copyright [year] [owner])
  • Files reference the LICENSE file or include abbreviated Apache 2.0 boilerplate headers
  • Any third-party or contributed code is properly attributed
📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8f6e4ce and 75f786d.

📒 Files selected for processing (1)
  • LICENSE (1 hunks)
🔇 Additional comments (1)
LICENSE (1)

1-201: LICENSE file correctly contains the canonical Apache License Version 2.0 text.

The file has 201 lines of text and is composed of the header (lines 1-3), the terms and conditions (lines 5-176) and the appendix (lines 178-201). All sections are present and match the canonical form from https://www.apache.org/licenses/LICENSE-2.0.txt.

Additionally, a NOTICE file should be present in the same directory, and each original source document should include a short license header at the top.

Merged via the queue into main with commit 301b4ff Dec 18, 2025
7 checks passed
@grahamc grahamc deleted the grahamc-patch-1 branch December 18, 2025 14:27
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.

License?

2 participants