Skip to content

Conversation

@amotl
Copy link
Member

@amotl amotl commented Jan 21, 2026

About

Validate removing a temporary workaround for an RTD API transition.

Preview

Unfortunately, the last mile can't be validated on the preview domain. It needs the real domain instead. 1
https://cratedb-guide--528.org.readthedocs.build/

References

Footnotes

  1. This means we need to swap it live, and roll back if it creates any problems.

@coderabbitai
Copy link

coderabbitai bot commented Jan 21, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

Added GitHub URL to the link checker ignore list in the Sphinx configuration file, and bumped the crate-docs-theme dependency version from >=0.42.0 to >=0.47.1.dev6 in the documentation requirements file.

Changes

Cohort / File(s) Summary
Sphinx Configuration
docs/conf.py
Added GitHub URL pattern (r"https://github.com/") to linkcheck_ignore list with a comment marker to exclude GitHub links from link validation checks.
Documentation Dependencies
docs/requirements.txt
Bumped crate-docs-theme version constraint from >=0.42.0 to >=0.47.1.dev6.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 GitHub links now skip the check,
Theme version takes a little trek,
From point-four to point-four-seven,
Docs configuration—pure magic, heaven! ✨

✨ 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 no-sphinx-build-compatibility

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.

@amotl amotl force-pushed the no-sphinx-build-compatibility branch from 45fa7f7 to 6f2c7de Compare January 21, 2026 06:32
@amotl amotl marked this pull request as ready for review January 21, 2026 06:45
@amotl amotl merged commit 6821ef4 into main Jan 21, 2026
2 of 3 checks passed
@amotl amotl deleted the no-sphinx-build-compatibility branch January 21, 2026 06:45
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