Fix: Index linkcheck warning

[bmunkholm]

About

make check (linkchecking) gave warnings and causing failed build in CI.

Preview

[coderabbitai]

Walkthrough

Moves the toctree index marker (index)= in docs/index.md and adds https://blog.nyrkio.com/ to linkcheck ignore lists in docs/conf.py. No other content or logic changes.

Changes

Cohort / File(s)	Summary
Documentation navigation docs/index.md	Relocated the (index)= toctree index marker from the start of the file to immediately before the CrateDB welcome content, changing the index anchor placement in the TOC.
Linkcheck configuration docs/conf.py	Added https://blog.nyrkio.com/ to linkcheck_ignore and linkcheck_anchors_ignore_for_url to skip 429/anchor link checks for that domain.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Intersphinx: Fix lost index link target #439 — Also adjusts the docs index anchor/placement in docs/index.md.
• Chore: Satisfy link checker #531 — Updates link-checker ignore rules in docs/conf.py (similar configuration changes).
• Chore: Ignore link checker errors on dzone.com, probably using a WAF now #208 — Modifies Sphinx docs linkcheck configuration to add domains to ignore lists.

Suggested labels

maintenance, cross linking

Suggested reviewers

• amotl
• kneth
• hlcianfagna

Poem

🐰 I hopped the index, soft and bright,
Placed it where the welcome's light,
Links spared from angry 429s,
I chew the docs and trim the vines,
Hooray — the table hops just right! 🎉

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: fixing linkcheck warnings by addressing an index-related issue in the documentation.
Description check	✅ Passed	The description is related to the changeset, explaining that linkcheck warnings caused CI build failures, which aligns with the configuration changes to ignore specific URLs.
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 docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch bm/fix-index

---

No actionable comments were generated in the recent review. 🎉

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[amotl]

🤷

[bmunkholm]

Yeah - not a problem IRL. Browsers handle fine.

Another problem related to building is this reoccuring rate-limiting for this article - it's taking forever to build:
(feature/index/index: line 108) ok https://blog.nyrkio.com/wp-content/uploads/2024/07/Nyrkio-comparison-of-CrateDB-and-MongoDB-with-TSBS-v2.pdf
-rate limited- https://blog.nyrkio.com/2024/07/11/timeseries-benchmark-on-cratedb-and-mongodb/ | sleeping...
-rate limited- https://blog.nyrkio.com/2024/07/11/timeseries-benchmark-on-cratedb-and-mongodb/ | sleeping...
-rate limited- https://blog.nyrkio.com/2024/07/11/timeseries-benchmark-on-cratedb-and-mongodb/ | sleeping...
-rate limited- https://blog.nyrkio.com/2024/07/11/timeseries-benchmark-on-cratedb-and-mongodb/ | sleeping...