Skip to content

Conversation

@amotl
Copy link
Member

@amotl amotl commented Jan 21, 2026

Just maintenance.

@coderabbitai
Copy link

coderabbitai bot commented Jan 21, 2026

Walkthrough

This PR adds three external documentation URLs (Astronomer, Confluent, and TensorFlow) to the link checker ignore list in the documentation configuration, preceded by a timeout comment. No functional changes are introduced.

Changes

Cohort / File(s) Summary
Documentation Configuration
docs/conf.py
Added three URLs to linkcheck_ignore list with timeout comment: Astronomer docs, Confluent docs, and TensorFlow docs

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

maintenance

Suggested reviewers

  • bmunkholm
  • zolbatar

Poem

🐰 A little hop through the docs we go,
Some links timeout, now watch them glow—
In the ignore list they find their rest,
Where patient checkers know what's best!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Chore: Satisfy link checker' is clear, concise, and directly describes the main change—adding URLs to the link checker ignore list in docs/conf.py.
Description check ✅ Passed The description 'Just maintenance.' is minimal but related to the changeset, which is maintenance work to satisfy the link checker by ignoring timeout-prone 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 fix-timeout-links

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 merged commit b28a790 into main Jan 21, 2026
3 of 4 checks passed
@amotl amotl deleted the fix-timeout-links branch January 21, 2026 13:14
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.

3 participants