Skip to content

Adopt Apache License 2.0 - #291

Merged
milind-soni merged 2 commits into
mainfrom
codex/apache-2-license
Aug 20, 2026
Merged

Adopt Apache License 2.0#291
milind-soni merged 2 commits into
mainfrom
codex/apache-2-license

Conversation

@milind-soni

@milind-soni milind-soni commented Aug 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • adopt Apache License 2.0 as the sole project license
  • record the unanimous contributor relicensing in NOTICE
  • ship LICENSE and NOTICE in every desktop distribution
  • update package metadata, README, and contribution terms

Validation

  • LICENSE matches the official Apache-2.0 text byte-for-byte
  • package.json and electron-builder.yml parse successfully
  • git diff --check passes

Relicensing

The project owner confirmed consent from all contributors whose copyright was incorporated before the transition.

Summary by CodeRabbit

  • Documentation
    • Updated licensing information throughout the project to Apache License 2.0.
    • Revised contribution guidance, README references, and legal notices to reflect the new license.
  • Chores
    • Updated application packaging to include the current license and notice documents.
    • Updated project metadata to identify Apache-2.0 licensing.

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 48ac4e3e-1222-4999-8de3-b00e9013bf4f

📥 Commits

Reviewing files that changed from the base of the PR and between f5115b5 and 714fa7f.

📒 Files selected for processing (3)
  • NOTICE
  • README.md
  • electron-builder.yml

📝 Walkthrough

Walkthrough

The project changes its primary license from MIT to Apache License 2.0. It updates contribution terms, attribution notices, README text, package metadata, and desktop build resources.

Changes

Licensing transition

Layer / File(s) Summary
License terms and attribution
LICENSE, CONTRIBUTING.md, NOTICE, README.md
The repository adopts Apache License 2.0 terms and updates contribution, attribution, and README licensing statements.
Package metadata and bundled notices
package.json, electron-builder.yml
Package metadata uses Apache-2.0. Desktop builds package LICENSE and NOTICE under licenses/ and remove LICENSE-MIT.

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

Suggested reviewers: kesleydavid

🚥 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.
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 and concisely describes the primary change: adopting Apache License 2.0.
Description check ✅ Passed The description explains the licensing change, rationale, validation, and contributor consent, but it omits the template headings and checklist.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/apache-2-license

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

@milind-soni
milind-soni merged commit 605eba0 into main Aug 20, 2026
5 of 6 checks passed
kargnas added a commit to kargnas/OpenMausBot that referenced this pull request Aug 20, 2026
main의 milind-soni#292(취소 시 승인 종료), milind-soni#264(리액션 팔레트), milind-soni#291(Apache
라이선스) 병합 충돌을 해결했다.

PATCH의 modelSelection 검증을 관대하게 바꿨다: 미확인 인스턴스도
persist되고(사전 엔진 구성·duplicate 시나리오), startTurn이 전송
시점에 409로 크게 실패한다. catalog 기반 모델 검증은 온라인
엔진에만 적용된다. 이에 따라 관련 e2e 기대치를 갱신했다.

Tested: pnpm typecheck, pnpm vitest run (132 files, 1311 passed, 12 skipped)

Confidence: high
Scope-risk: moderate
Reversability: moderate
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