Skip to content

Fix typo: Seperation -> Separation in deployment strategy doc#10986

Open
vipooshanb wants to merge 1 commit intowso2:masterfrom
vipooshanb:fix-typo-deployment-strategy
Open

Fix typo: Seperation -> Separation in deployment strategy doc#10986
vipooshanb wants to merge 1 commit intowso2:masterfrom
vipooshanb:fix-typo-deployment-strategy

Conversation

@vipooshanb
Copy link

@vipooshanb vipooshanb commented Mar 25, 2026

Purpose

Fix a spelling mistake in choosing-your-deployment-strategy.md.

Changes

  • Corrected typo: SeperationSeparation
  • Location: Scenario 5 - "Fully Distributed with Key Manager Separation"

Type

Documentation fix (typo correction)

Impact

No functional changes. Improves documentation clarity and correctness.

Summary by CodeRabbit

  • Documentation
    • Corrected a spelling error in the deployment strategy guide.

Copilot AI review requested due to automatic review settings March 25, 2026 11:28
@vipooshanb vipooshanb requested a review from chamilaadhi as a code owner March 25, 2026 11:28
@coderabbitai
Copy link

coderabbitai bot commented Mar 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 77dd35ad-8b88-407d-9a3b-4824e632591b

📥 Commits

Reviewing files that changed from the base of the PR and between bcf383b and 3235832.

📒 Files selected for processing (1)
  • en/docs/get-started/choosing-your-deployment-strategy.md

Walkthrough

A single spelling correction in deployment strategy documentation, changing "Seperation" to "Separation" in a link label within Scenario 5. No functional changes or code alterations.

Changes

Cohort / File(s) Summary
Documentation Spelling Fix
en/docs/get-started/choosing-your-deployment-strategy.md
Corrected spelling of "Separation" in Pattern 4 link text within the "Ultimate Scalability & Isolation" scenario.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

🐰 A typo hopped away, now spelling's spry,
"Seperation" now "Separation"—oh my oh my!
One letter fixed, precision restored,
Documentation's path now gently scored! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description provides basic information about the fix, but lacks comprehensive coverage of most required template sections. While the purpose and change are clear, consider whether required sections like Goals, Approach, Release notes, and Documentation links should be included for consistency with the template.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change: fixing a typo in the deployment strategy documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes a spelling mistake in the deployment strategy documentation to improve clarity and correctness.

Changes:

  • Corrected “Seperation” → “Separation” in Scenario 5’s Pattern 4 link text

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants