Skip to content

Conversation

@lio-p
Copy link

@lio-p lio-p commented Oct 29, 2025

Summary by CodeRabbit

  • Chores
    • Added ClickHouse to the engineering blogs collection with RSS feed and blog category links for content discovery.

@coderabbitai
Copy link

coderabbitai bot commented Oct 29, 2025

Walkthrough

This pull request adds ClickHouse to a curated list of engineering blogs. A new entry is added to README.md under the C companies section, and a corresponding RSS feed entry is added to the engineering_blogs.opml file.

Changes

Cohort / File(s) Summary
ClickHouse Blog Addition
README.md, engineering_blogs.opml
Added ClickHouse engineering blog entry (https://clickhouse.com/blog?category=engineering) to README under C companies section and added corresponding RSS feed entry (https://clickhouse.com/rss.xml) to OPML file.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Both changes are straightforward additions to static data structures (a markdown list and an XML outline).
  • No logic, configuration, or structural modifications involved.

Poem

🐰 A whisker-twitch of joy, hooray!
ClickHouse joins the blogs today,
In alphabets from A to Z,
Where engineering tales run free. ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title "Add Clickhouse engineering blog" directly and accurately describes the primary change in this pull request. The changeset adds ClickHouse as a new entry to both README.md and engineering_blogs.opml files, which is precisely what the title conveys. The title is concise at four words, specific enough for a developer scanning history to understand the change at a glance, and avoids vague or generic language.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 50eab27 and 4bf5dec.

📒 Files selected for processing (2)
  • README.md (1 hunks)
  • engineering_blogs.opml (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
README.md

62-62: Bare URL used

(MD034, no-bare-urls)

🔇 Additional comments (2)
README.md (1)

62-62: ✓ ClickHouse entry verified.

The ClickHouse blog URL is accessible (HTTP 200). The entry is correctly positioned alphabetically between Clever and Cloudera, follows the established format, and is ready to merge.

engineering_blogs.opml (1)

35-35: LGTM! ClickHouse RSS entry properly formatted.

All verification checks passed:

  • ✓ OPML entry correctly structured with valid attributes
  • ✓ Alphabetically positioned between "Clever" and "Cloudera"
  • ✓ htmlUrl matches README.md entry exactly (https://clickhouse.com/blog?category=engineering)
  • ✓ RSS feed accessible (HTTP 200) with valid RSS 2.0 structure
  • ✓ Blog URL accessible (HTTP 200)

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.

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.

1 participant