Skip to content

fix: remove duplicated boilerplate from LICENSE appendix - #14

Merged
NX1X merged 1 commit into
mainfrom
fix/license-copyright
Jul 25, 2026
Merged

fix: remove duplicated boilerplate from LICENSE appendix#14
NX1X merged 1 commit into
mainfrom
fix/license-copyright

Conversation

@NX1X

@NX1X NX1X commented Jul 25, 2026

Copy link
Copy Markdown
Owner

Summary

LICENSE carried the closing Apache notice twice: the unfilled Copyright [yyyy] [name of copyright owner] template, immediately followed by a second copy naming the owner. Someone had appended a filled-in notice rather than filling in the template, leaving 216 lines where the canonical Apache License 2.0 is 202.

The appendix now appears once, reading Copyright 2025-2026 NX1X. The rest of the file is byte-identical to the canonical text from apache.org.

- Copyright [yyyy] [name of copyright owner]
- ...13 lines of duplicated boilerplate...
  Copyright 2025-2026 NX1X

Verification

  • LICENSE is now 202 lines, matching the canonical Apache License 2.0
  • Lines 1-187 diff clean against https://www.apache.org/licenses/LICENSE-2.0.txt
  • No [yyyy] or [name of copyright owner] placeholder remains
  • Exactly one copyright line
  • GitHub already detected Apache-2.0 before this change and continues to; the ownership statement is now unambiguous

Folds into the existing unreleased 1.5.1 entry - no version bump needed.

Summary by CodeRabbit

  • Documentation
    • Updated the changelog to note the corrected license appendix.
    • Updated the license with the project’s current copyright attribution.

The file carried the closing Apache notice twice - the unfilled
'Copyright [yyyy] [name of copyright owner]' template, followed by a
second copy naming the owner. The appendix now appears once, reading
'Copyright 2025-2026 NX1X', and the file is otherwise byte-identical to
the canonical Apache License 2.0 text (202 lines).
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 25, 2026

Copy link
Copy Markdown

Deploying easyconvert-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: b56f1f2
Status: ✅  Deploy successful!
Preview URL: https://f13fd243.easyconvert-website.pages.dev
Branch Preview URL: https://fix-license-copyright.easyconvert-website.pages.dev

View logs

@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@coderabbitai

coderabbitai Bot commented Jul 25, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b10ba9e7-d1b2-494b-8cc3-c694b49d27fc

📥 Commits

Reviewing files that changed from the base of the PR and between 5ee1063 and b56f1f2.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • LICENSE
💤 Files with no reviewable changes (1)
  • LICENSE

📝 Walkthrough

Walkthrough

The Apache license appendix now uses Copyright 2025-2026 NX1X, and CHANGELOG.md records the correction for version 1.5.1.

Changes

License correction

Layer / File(s) Summary
Update license attribution and changelog
LICENSE, CHANGELOG.md
The license appendix replaces template placeholders with the NX1X copyright line, and the changelog documents the correction.

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

Poem

A bunny hops through license text,
Finds old placeholders, lays them to rest.
NX1X now shines in line,
The changelog records the sign.
One neat appendix, crisp and bright!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: cleaning up duplicated boilerplate in the LICENSE appendix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

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

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

@NX1X
NX1X merged commit 483aaf6 into main Jul 25, 2026
10 checks passed
@NX1X
NX1X deleted the fix/license-copyright branch July 25, 2026 11:50
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