skills.rest
codacy-open-source-projects-scans
codacy-open-source-projects-scansOFFICIAL
Explore all skills by this author →
GitHub

review

Review ClickHouse pull requests for correctness, safety, performance, and compliance in C++ code.

1|Updated Jul 29, 2024
Search Tags:#code review#performance#compliance#c++#clickhouse#pr analysis
One-click install
npx skills add https://github.com/codacy-open-source-projects-scans/clickhouse --skill review-codacy-open-source-projects-scans
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/codacy-open-source-projects-scans/clickhouse/tree/main/.claude/skills/review
Command: npx skills add https://github.com/codacy-open-source-projects-scans/clickhouse --skill review-codacy-open-source-projects-scans

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes scripts (resource) and references (resource) components.

What problem does it solve?

This Skill automates the rigorous code review process for ClickHouse Pull Requests, ensuring correctness, safety, performance, and compliance with project standards.

Core Features & Use Cases

  • Automated Code Analysis: Performs a strict, high-signal review of C++ code, focusing on correctness, resource management, concurrency, and performance.
  • Compliance Checks: Verifies adherence to ClickHouse-specific rules like deletion logging, serialization versioning, and experimental gating.
  • Use Case: When a developer submits a new feature for ClickHouse, this Skill can be invoked to provide an immediate, detailed review, flagging potential issues before human reviewers even start.

Quick Start

Use the review skill to analyze pull request number 12345.

Frequently Asked Questions about review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review ClickHouse pull requests for correctness and performance?▼

Review ClickHouse pull requests by analyzing C++ core code, query execution, and storage components against best practices to identify logic errors, resource leaks, concurrency issues, and performance regressions.

What compliance checks are needed for ClickHouse C++ code?▼

ClickHouse C++ code compliance checks verify adherence to project-specific rules including deletion logging, serialization versioning, and experimental gating requirements using a strict, high-signal methodology.

Can I automate PR analysis for ClickHouse server components?▼

Automate ClickHouse server component PR analysis by invoking a review process that flags potential correctness, safety, and concurrency issues before human reviewers start.

Does this code review method identify resource leaks in C++ query execution?▼

C++ code review for ClickHouse query execution identifies resource leaks, concurrency issues, and performance regressions by analyzing the code against industry best practices and ClickHouse-specific rules.

What is the best way to check ClickHouse serialization versioning compliance?▼

Check ClickHouse serialization versioning compliance by running an automated review that verifies C++ core code and server components adhere to strict project standards and experimental gating requirements.

Why does my ClickHouse PR analysis fail on deletion logging requirements?▼

ClickHouse PR analysis fails on deletion logging when the C++ core code does not adhere to strict compliance rules for deletion logging, serialization versioning, or experimental gating requirements.

Related Skills

Explore More Software Engineering Skills→
susamnsusamn

pr-review

Compile GitHub PR metadata and diffs into structured LLM code review prompts.

Community
Intermediate
aide-familyaide-family
253

code-review

Review code for bugs with file and line number precision.

Official
Intermediate
ErnestPenaJrErnestPenaJr

csharp-validator

Validate C# code quality for .NET applications using Roslyn analyzers.

Community
Intermediate
bejranondabejranonda
27

web-validation

Detect JavaScript errors and collect browser diagnostics from web pages.

Community
Advanced