Skip to content

chore: include LICENSE in final crate tarball#657

Merged
alexhancock merged 1 commit intomodelcontextprotocol:mainfrom
r0x0d:include-license-in-final-crate
Feb 13, 2026
Merged

chore: include LICENSE in final crate tarball#657
alexhancock merged 1 commit intomodelcontextprotocol:mainfrom
r0x0d:include-license-in-final-crate

Conversation

@r0x0d
Copy link
Contributor

@r0x0d r0x0d commented Feb 13, 2026

Motivation and Context

This is required for packaging in distributions like Fedora and probably others too

How Has This Been Tested?

$ cargo package --list | grep LICENSE
LICENSE

Breaking Changes

No need. This is mainly regarding the crate published in crates.io

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@github-actions github-actions bot added T-dependencies Dependencies related changes T-config Configuration file changes T-macros Macro changes labels Feb 13, 2026
@alexhancock
Copy link
Contributor

@r0x0d the commit message just needs to be adjusted to adhere to conventional commit format. I did add a pre-commit linter for this, but let me know if it's not working for you.

@r0x0d
Copy link
Contributor Author

r0x0d commented Feb 13, 2026

@r0x0d the commit message just needs to be adjusted to adhere to conventional commit format. I did add a pre-commit linter for this, but let me know if it's not working for you.

Oh, right! I saw that, and I wanted to go back to this, but forgot due to other activities. I will fix it now.

Required for packaging in distributions such as Fedora and others.

Verified with:
$ cargo package --list | grep LICENSE
@r0x0d r0x0d force-pushed the include-license-in-final-crate branch from ddf17e2 to 2dd308f Compare February 13, 2026 21:01
@r0x0d r0x0d changed the title Include LICENSE in the final crate tarball chore: include LICENSE in final crate tarball Feb 13, 2026
@alexhancock alexhancock merged commit 453032f into modelcontextprotocol:main Feb 13, 2026
11 checks passed
@r0x0d r0x0d deleted the include-license-in-final-crate branch February 13, 2026 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-config Configuration file changes T-dependencies Dependencies related changes T-macros Macro changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants