Skip to content

fix: ship LICENSE in rmcp and rmcp-macros crates - #1342

Merged
DaleSeo merged 2 commits into
modelcontextprotocol:mainfrom
Steeve-Crypto:fix/package-license
Oct 14, 2026
Merged

DaleSeo merged 2 commits into
modelcontextprotocol:mainfrom
Steeve-Crypto:fix/package-license

Conversation

@Steeve-Crypto

Copy link
Copy Markdown
Contributor

Closes #1335

Adds a LICENSE symlink in each crate dir so cargo package includes the root LICENSE again, without bringing back license-file and its Cargo warning.

Test plan: cargo package --list -p rmcp and -p rmcp-macros both list LICENSE; new cargo test -p rmcp --test test_package_license passes; fmt and clippy clean.

@Steeve-Crypto
Steeve-Crypto requested a review from a team as a code owner October 11, 2026 22:17
@github-actions github-actions Bot added T-test Testing related changes T-macros Macro changes labels Oct 11, 2026
Comment thread crates/rmcp/tests/test_package_license.rs
Comment thread crates/rmcp/tests/test_package_license.rs
@Steeve-Crypto

Copy link
Copy Markdown
Contributor Author

Applied both suggestions, thanks

@github-actions github-actions Bot added T-dependencies Dependencies related changes T-CI Changes to CI/CD workflows and configuration T-config Configuration file changes labels Oct 13, 2026
@DaleSeo
DaleSeo merged commit e957a53 into modelcontextprotocol:main Oct 14, 2026
24 checks passed
@github-actions github-actions Bot mentioned this pull request Oct 12, 2026
This was referenced Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-CI Changes to CI/CD workflows and configuration T-config Configuration file changes T-dependencies Dependencies related changes T-macros Macro changes T-test Testing related changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rmcp and rmcp-macros crates no longer include LICENSE (regression since 1.3.0)

2 participants