Skip to content

Relicense to Apache-2.0 - #12

Merged
Moge800 merged 1 commit into
mainfrom
claude/apache-license
Jul 22, 2026
Merged

Relicense to Apache-2.0#12
Moge800 merged 1 commit into
mainfrom
claude/apache-license

Conversation

@Moge800

@Moge800 Moge800 commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Summary

The LICENSE file is already Apache-2.0 (a1af65b), but pyproject.toml still declared MIT — this fixes that inconsistency so the package metadata matches.

  • pyproject.toml: license = { text = "MIT" }{ text = "Apache-2.0" }

Why

Aligns the wrapper's license with the bundled gomc-rest server, which is Apache-2.0 as of v1.5.0. Since the wheels redistribute that binary, matching licenses removes the MIT-wrapper / Apache-payload mismatch.

Verified

Built the wheel locally and inspected METADATA:

License: Apache-2.0
License-File: LICENSE

The Apache LICENSE text is bundled under .dist-info/licenses/LICENSE. import gomc_rest still works.

🤖 Generated with Claude Code


Generated by Claude Code

The LICENSE file was already switched to Apache-2.0 (a1af65b); align the
package metadata so it matches. This also makes the wrapper's license
consistent with the bundled gomc-rest server (Apache-2.0 since v1.5.0),
so redistributing that binary inside the wheels no longer mixes licenses.

Verified: built wheel METADATA now reports 'License: Apache-2.0' and
bundles the LICENSE under .dist-info/licenses/.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SQbVkPEprK2ebcjck5ouUy
Copilot AI review requested due to automatic review settings July 22, 2026 00:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Moge800
Moge800 merged commit 7536905 into main Jul 22, 2026
2 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.

3 participants