Skip to content

docs: fix typo protocal -> protocol - #2816

Open
vaibhav8a wants to merge 1 commit into
cloudfoundry:mainfrom
vaibhav8a:fix-typo-protocol
Open

docs: fix typo protocal -> protocol#2816
vaibhav8a wants to merge 1 commit into
cloudfoundry:mainfrom
vaibhav8a:fix-typo-protocol

Conversation

@vaibhav8a

Copy link
Copy Markdown

One-word typo fix in src/bosh-monitor/lib/bosh/monitor/plugins/README.md:21 — a table row label reading protocalprotocol.

@linux-foundation-easycla

Copy link
Copy Markdown

CLA Not Signed

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 774a47ad-8c6b-4b24-8b42-bb73de0baa37

📥 Commits

Reviewing files that changed from the base of the PR and between 1172d3b and ef227ed.

📒 Files selected for processing (1)
  • src/bosh-monitor/lib/bosh/monitor/plugins/README.md

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.


Walkthrough

The Consul plugin README now documents the HTTP configuration option as protocol instead of protocal.

Merge Risk: ⚪ Minimal · up to ef227

This is a localized documentation typo fix with no actionable merge-blocking risk remaining.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description accurately identifies the typo fix and the affected README file.
Title check ✅ Passed The title clearly and concisely describes the typo fix in the documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 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.

@rkoster

rkoster commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

@vaibhav8a please address: #2816 (comment) so we can look into this PR.

@aramprice aramprice moved this from Pending Merge | Prioritized to Waiting for Changes | Open for Contribution in Foundational Infrastructure Working Group Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting for Changes | Open for Contribution

Development

Successfully merging this pull request may close these issues.

3 participants