Skip to content

chore: remove project-wide license declarations - #32

Merged
builtbyrishabh merged 1 commit into
mainfrom
chore/remove-project-license
Sep 30, 2026
Merged

builtbyrishabh merged 1 commit into
mainfrom
chore/remove-project-license

Conversation

@builtbyrishabh

@builtbyrishabh builtbyrishabh commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

AnimeRoom currently grants project-wide MIT permissions. Remove the root license file and README license badge/declaration, retaining Rishabh Singh’s copyright attribution and clarifying that existing licensed copies keep their original permissions.

Format the edited README with the existing Prettier configuration; its formatting was the cause of the previous main-branch CI failure.

Validation: pnpm typecheck, pnpm lint, pnpm format:check, pnpm test (25 passed), and pnpm build all passed. Lint and build used the repository’s CI placeholder environment. git diff --check passed; no runtime code or dependencies changed.

pnpm audit --audit-level high fails on the existing dependencies (43 high and 2 critical findings). The package manifest and lockfile are identical to main; this licensing/documentation change does not add or update any dependencies.

@vercel

vercel Bot commented Sep 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
anime-room Building Building Preview Sep 30, 2026 5:24pm UTC

@coderabbitai

coderabbitai Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 03e3a472-91fc-4c00-b644-198e5038478a

📥 Commits

Reviewing files that changed from the base of the PR and between 34bfcd4 and 3fa58f8.

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

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

📜 Recent review details
⏰ Context from checks skipped due to timeout. (1)
  • GitHub Check: verify
🔇 Additional comments (1)
README.md (1)

79-93: LGTM!

Also applies to: 154-163, 165-171


📝 Summary

Summary by CodeRabbit

  • Documentation
    • Updated the README’s copyright notice to state that this revision does not offer a project-wide license, while previously licensed copies retain their original permissions and third-party components remain subject to their own licenses.
    • Removed the MIT license badge and the project’s LICENSE file.

Walkthrough

The pull request removes the MIT license file and README badge. It replaces the README license section with a copyright notice and reformats the technology-stack and scripts tables without changing their listed content.

Changes

License notice update

Layer / File(s) Summary
License notice and README edits
LICENSE, README.md
The README replaces its MIT license references with a copyright notice covering this revision, previously licensed copies, and third-party components. The MIT license file is removed. The technology-stack and scripts tables are reformatted without changes to their content.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 3fa58

The README now states that no project-wide license is offered for this revision, with no conflicting project-license claim found. No actionable merge risk remains.

Architecture Summary

Architecture risk: 🔵 Low · up to 3fa58

The change affects 2 systems.

Changed systems: LICENSE, README.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — LICENSE (service) was modified; 1 changed file maps to changed impact.
  • observed — README.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in LICENSE: The LICENSE file’s MIT License terms and copyright notice were removed.
  • observed — Modified behavior in README.md: The MIT license badge was removed from the header.
  • observed — Modified behavior in README.md: The technology-stack table is reformatted with narrower column spacing; its listed areas and choices are unchanged.
  • observed — Modified behavior in README.md: The scripts table is reformatted with narrower column spacing; its commands and descriptions are unchanged.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title clearly identifies the main change: removal of project-wide license declarations.
Description check ✅ Passed The description directly explains the license changes, retained copyright attribution, README formatting, and validation results.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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 reads the notice with care,
Then hops past the badge that is no longer there.
The tables sit neat, their contents the same,
While license notes now state their claim.
One last soft thump, and the page is clear.

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

@builtbyrishabh
builtbyrishabh merged commit 8c6bd2c into main Sep 30, 2026
2 of 4 checks passed
@builtbyrishabh
builtbyrishabh deleted the chore/remove-project-license branch September 30, 2026 17:27

This branch was successfully deployed

1 active deployment
Preview — 3fa58f82 Deployed Sep 30, 2026 by vercel[bot]
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