Skip to content

chore(license): normalize Apache 2.0 notice - #15

Merged
yordis merged 1 commit into
mainfrom
yordis/chore-normalize-apache-license
Sep 17, 2026
Merged

yordis merged 1 commit into
mainfrom
yordis/chore-normalize-apache-license

Conversation

@yordis

@yordis yordis commented Sep 17, 2026

Copy link
Copy Markdown
Member
  • Ensures downstream users receive the complete standard Apache 2.0 licensing terms while preserving the original MIT attribution.

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@cursor

cursor Bot commented Sep 17, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation-only LICENSE text updates with no effect on application behavior or dependencies.

Overview
Adds the standard Apache 2.0 appendix (application instructions plus boilerplate notice) to the root LICENSE and the matching LICENSE files under trogon-eventstore, trogon-eventstore-extras, and trogon-eventstore-macros, so each copy ends with Copyright 2026 Straw Hat, LLC and the usual short Apache 2.0 header text after END OF TERMS AND CONDITIONS.

No runtime or build code changes; inherited MIT terms remain in LICENSES/MIT.txt and are not modified in this diff.

Reviewed by Cursor Bugbot for commit ce490a8. Bugbot is set up for automated code reviews on this repo. Configure here.

@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 808896bf-c8ae-4111-a78e-b560fa608119

📥 Commits

Reviewing files that changed from the base of the PR and between 4b0bfa8 and ce490a8.

📒 Files selected for processing (4)
  • LICENSE
  • trogon-eventstore-extras/LICENSE
  • trogon-eventstore-macros/LICENSE
  • trogon-eventstore/LICENSE

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The pull request adds the standard Apache License 2.0 appendix to four license files. Each appendix includes application instructions and a 2026 copyright notice for Straw Hat, LLC.

Changes

License appendices

Layer / File(s) Summary
Add Apache License appendices
LICENSE, trogon-eventstore-*/LICENSE
The four license files now include Apache License 2.0 application instructions, the standard notice text, a 2026 Straw Hat, LLC copyright attribution, and licensing disclaimers.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to ce490

The license notice update is ready to merge; inherited MIT attribution remains available with each package.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: normalizing the Apache 2.0 license notice.
Description check ✅ Passed The description directly relates to the license changes and states their purpose for downstream users.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch yordis/chore-normalize-apache-license

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.

❤️ Share

A rabbit checks each license line
The appendix fits in place just fine
Straw Hat’s notice marks the year
Apache terms now all appear
Four files rest, complete and clear

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

@yordis
yordis merged commit 243c4e5 into main Sep 17, 2026
8 checks passed
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.

1 participant