review-mr

Verify merge requests for commit format, code quality, and testing compliance.

38|15|Updated Jul 26, 2023
One-click install
npx skills add https://github.com/sisense/compose-sdk-monorepo --skill review-mr-sisense
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review-mr
Source: https://github.com/sisense/compose-sdk-monorepo/tree/main/.claude/skills/review-mr
Command: npx skills add https://github.com/sisense/compose-sdk-monorepo --skill review-mr-sisense

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill helps ensure MR artifacts conform to established conventions and quality standards by checking commit formats, absence of Sisense internals in source, and required testing criteria.

Core Features & Use Cases

  • Commit message validation: verifies header length, presence of Jira/ONYX ticket references, lowercase after the colon, and imperative language.
  • Source integrity checks: flags internal references or patterns that violate project policies in code.
  • Code quality guidance: enforces general best practices and repository-specific rules for maintainable changes.
  • Testing requirements: outlines and enforces the necessary testing expectations for MRs.

Quick Start

Review the current MR to verify conformance with commit formats, code quality rules, and testing requirements.

Frequently Asked Questions about review-mr

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

FAQPage Schema
How do I validate commit messages in a merge request for Jira ticket references?

Merge request validation checks commit header length, presence of Jira or ONYX ticket references, lowercase text after the colon, and imperative language. This ensures your merge requests comply with project conventions and pass quality audits.

What merge request quality checks should I enforce for feature and hotfix branches?

Merge request quality checks for feature and hotfix branches should verify commit message formats, scan changed files for prohibited internal references, and enforce repository-specific code-quality and testing standards. This maintains source integrity and compliance across the repository.

How do I scan source code to prohibit Sisense internal references in merge requests?

To prohibit Sisense internal references in source code, you can apply an MR review that flags internal references or patterns violating project policies in changed files. This identifies deviations from source integrity rules during the merge audit process.

What is the best way to automate code review and linting for merge request compliance?

Automating code review for merge request compliance involves auditing MR artifacts against established conventions, checking commit formats, enforcing code quality rules, and outlining required testing criteria. This reports deviations from project standards automatically without manual inspection.

Does merge request auditing work with Git and Jira workflows?

Yes, merge request auditing works with Git and Jira workflows by verifying that commit messages contain valid Jira ticket references and applying quality standards to feature and hotfix branches. It reports any deviations from these integration conventions directly.