Skip to content

Update link in installation guide for node placement#250

Merged
DTTerastar merged 1 commit intoESPresense:mainfrom
14790897:patch-1
Feb 1, 2026
Merged

Update link in installation guide for node placement#250
DTTerastar merged 1 commit intoESPresense:mainfrom
14790897:patch-1

Conversation

@14790897
Copy link
Copy Markdown
Contributor

@14790897 14790897 commented Jan 17, 2026

the link is broken

Summary by CodeRabbit

Release Notes

  • Documentation
    • Fixed internal navigation links in the installation guide for improved documentation consistency and accessibility.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 17, 2026

Warning

Rate limit exceeded

@DTTerastar has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 3 minutes and 1 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Walkthrough

Two internal documentation links in companion/installation.md were updated. The anchor references were changed from /companion/configuration#node-placement--configuration to /companion/configuration#node-placement, affecting the Node Placement Guide navigation targets.

Changes

Cohort / File(s) Summary
Documentation Links
companion/installation.md
Updated two internal reference anchors for Node Placement Guide from #node-placement--configuration to #node-placement

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A rabbit hops through links so true,
Fixing anchors, old then new,
From --configuration we break free,
Now #node-placement guides you and me! ✨

🚥 Pre-merge checks | ✅ 3
✅ 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 accurately and concisely describes the main change: updating a broken link in the installation guide for node placement.
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
  • Post copyable unit tests in a comment

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.

@DTTerastar DTTerastar force-pushed the main branch 5 times, most recently from f5ebb1e to 7048e05 Compare January 30, 2026 03:43
@DTTerastar DTTerastar merged commit a7ef573 into ESPresense:main Feb 1, 2026
1 check passed
@DTTerastar
Copy link
Copy Markdown
Contributor

Thanks!

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