Skip to content

fix: Remove unnecessary doc-cfg#661

Open
Evian-Zhang wants to merge 2 commits intomodelcontextprotocol:mainfrom
Evian-Zhang:remove-doc-cfg
Open

fix: Remove unnecessary doc-cfg#661
Evian-Zhang wants to merge 2 commits intomodelcontextprotocol:mainfrom
Evian-Zhang:remove-doc-cfg

Conversation

@Evian-Zhang
Copy link

Motivation and Context

After the doc_auto_cfg feature has been merged into doc_cfg feature, there is no need to annotate #[cfg_attr(docsrs, doc(cfg(feature = "xxxx")))], it will be automatically identified and rendered in the rustdoc.

How Has This Been Tested?

Tested locally with

RUSTDOCFLAGS="--cfg docsrs" cargo +nightly doc --no-deps --open

Breaking Changes

No

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-core Core library changes T-transport Transport layer changes labels Feb 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-core Core library changes T-transport Transport layer changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant