Skip to content

Conversation

@Pouyanpi
Copy link
Collaborator

Description

  • Update git-cliff configuration to skip CI-related commits from changelog generation
  • Separate the combined ^chore|^ci pattern into distinct rules for better control

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Nov 28, 2025

Greptile Overview

Greptile Summary

Updated git-cliff configuration to exclude CI-related commits from changelog generation while preserving chore commits.

  • Split the combined ^chore|^ci regex pattern into three separate rules for better control
  • Added ^ci pattern to skip commits starting with "ci"
  • Added \\(ci\\) pattern to skip commits with ci scope like chore(ci): or fix(ci):
  • Kept ^chore pattern to continue grouping non-CI chore commits in the changelog

The change correctly preserves the existing skip rules for release preparation, dependencies, and PR-related chores, while adding targeted exclusions for CI commits.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The change is a simple, well-scoped configuration update that correctly refactors the commit parser rules. The regex patterns are valid and have been verified against the actual commit history. The separation of combined patterns into distinct rules improves maintainability without affecting other functionality.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
cliff.toml 5/5 Split combined chore/CI pattern into separate rules to exclude CI commits from changelog while keeping chore commits

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Git as Git Repository
    participant Cliff as git-cliff
    participant CL as Changelog

    Dev->>Git: Commit with message
    Note over Git: Various commit types:<br/>ci:, chore(ci):, fix(ci):,<br/>chore:, feat:, fix:, etc.
    
    Git->>Cliff: Parse commits for changelog
    
    Cliff->>Cliff: Check commit_parsers rules (in order)
    
    alt Commit matches ^ci
        Cliff->>Cliff: Skip (line 82)
    else Commit matches \(ci\)
        Cliff->>Cliff: Skip (line 83)
    else Commit matches ^chore
        Cliff->>Cliff: Group under "Miscellaneous Tasks" (line 84)
    else Other patterns
        Cliff->>Cliff: Process according to other rules
    end
    
    Cliff->>CL: Generate changelog
    Note over CL: CI commits excluded<br/>Chore commits included
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@Pouyanpi Pouyanpi self-assigned this Nov 28, 2025
@codecov
Copy link

codecov bot commented Nov 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Pouyanpi Pouyanpi merged commit 72c8ec0 into develop Nov 28, 2025
7 checks passed
@Pouyanpi Pouyanpi deleted the chore/exclude-ci-from-changelog branch November 28, 2025 10:06
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