Skip to content

docs: remove duplicate line in MCP authentication documentation#982

Merged
rapids-bot[bot] merged 1 commit intoNVIDIA:release/1.3from
willkill07:wkk_fix-mcp-auth-duplicate
Oct 14, 2025
Merged

docs: remove duplicate line in MCP authentication documentation#982
rapids-bot[bot] merged 1 commit intoNVIDIA:release/1.3from
willkill07:wkk_fix-mcp-auth-duplicate

Conversation

@willkill07
Copy link
Member

@willkill07 willkill07 commented Oct 13, 2025

Description

Closes

By Submitting this PR I confirm:

  • I am familiar with the Contributing Guidelines.
  • We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license.
    • Any contribution which contains commits that are not Signed-Off will not be accepted.
  • When the PR is ready for review, new or existing tests cover these changes.
  • When the PR is ready for review, the documentation is up to date with these changes.

Summary by CodeRabbit

  • Documentation
    • Cleaned up the Authentication Best Practices section for MCP authentication by removing a duplicate header line.
    • Improved readability and reduced redundancy without altering the existing guidance or structure.
    • No functional or behavioral changes; content remains the same, just clearer presentation.

Signed-off-by: Will Killian <wkillian@nvidia.com>
@willkill07 willkill07 requested a review from a team as a code owner October 13, 2025 13:38
@coderabbitai
Copy link

coderabbitai bot commented Oct 13, 2025

Walkthrough

Removed a duplicate header line in the Authentication Best Practices section of the MCP authentication documentation. No other content or structure changed.

Changes

Cohort / File(s) Summary
Documentation cleanup
docs/source/workflows/mcp/mcp-auth.md
Deleted a duplicated header line in the "Authentication Best Practices" section; no other edits.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

doc

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title “docs: remove duplicate line in MCP authentication documentation” clearly and succinctly describes the main change, uses imperative mood, and stays within the 72 character limit while providing enough context for readers.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d91fd6c and 6680bd4.

📒 Files selected for processing (1)
  • docs/source/workflows/mcp/mcp-auth.md (0 hunks)
💤 Files with no reviewable changes (1)
  • docs/source/workflows/mcp/mcp-auth.md

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai bot added the doc Improvements or additions to documentation label Oct 13, 2025
@willkill07 willkill07 added the non-breaking Non-breaking change label Oct 13, 2025
@willkill07 willkill07 self-assigned this Oct 14, 2025
Copy link
Contributor

@AnuradhaKaruppiah AnuradhaKaruppiah left a comment

Choose a reason for hiding this comment

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

Thx for this fixup @willkill07

@willkill07
Copy link
Member Author

/merge

1 similar comment
@willkill07
Copy link
Member Author

/merge

@rapids-bot rapids-bot bot merged commit daca515 into NVIDIA:release/1.3 Oct 14, 2025
22 of 23 checks passed
@willkill07 willkill07 deleted the wkk_fix-mcp-auth-duplicate branch October 23, 2025 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Improvements or additions to documentation non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants