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
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