docs: fix broker container reference and producer typos#10449
docs: fix broker container reference and producer typos#10449gaoxiaolei-s59 wants to merge 1 commit into
Conversation
RongtongJin
left a comment
RongtongJin
left a comment
There was a problem hiding this comment.
Thanks for the fix. Before we move this forward, could you please open a corresponding issue that describes the problem this PR addresses and link it here?
Please also update the PR description to reference that issue, for example with Fixes #<issue-number>. We generally prefer to track even small fixes through a linked issue so the context and rationale are clear.
|
Thanks for the reminder. I opened #10451 to describe the documentation problems and updated this PR description with |
oss-sentinel-ai
left a comment
oss-sentinel-ai
left a comment
There was a problem hiding this comment.
Review: Approved ✅
PR: #10449 — docs: fix broker container reference and producer typos
Type: Documentation fix (2 files, +5/-6)
Assessment
Clean documentation fix addressing two issues:
- BrokerContainer.md: Updated config file path to reference existing sample
broker-container1.conf - API_Reference_DefaultMQProducer.md: Fixed typos (
ca quickly,Blow is,create a produce,rcpHook)
Verdict
✅ Straightforward documentation improvement. Fixes #10451.
🤖 Automated review by oss-sentinel-ai
3 participants
Which Issue(s) This PR Fixes
Fixes #10451
Brief Description
Fix two documentation issues:
distribution/conf/container/2container-2m-2s/broker-container1.conf.DefaultMQProducerAPI reference, includingca quickly,Blow is,create a produce, andrcpHook.How Did You Test This Change?
git diff --check HEAD~1..HEADrg -n "ca quickly|Blow is|rcpHook|distribution/conf/container/broker-container\.conf|create a produce with" docs/cn/BrokerContainer.md docs/en/client/java/API_Reference_DefaultMQProducer.md || truetest -f distribution/conf/container/2container-2m-2s/broker-container1.confDocumentation-only change.