Skip to content

ci: consolidate PR workflows and integrate releaseforge for auto-versioning#4

Merged
machado144 merged 1 commit intomainfrom
chore/add-pr-title-validation
Mar 8, 2026
Merged

ci: consolidate PR workflows and integrate releaseforge for auto-versioning#4
machado144 merged 1 commit intomainfrom
chore/add-pr-title-validation

Conversation

@machado144
Copy link
Contributor

Summary

  • Add PR title validation workflow to enforce conventional commit format
  • Ensures squash merge commits follow conventional commits for automated version bumps

Test plan

  • Open a PR with invalid title and verify it fails
  • Open a PR with valid conventional commit title and verify it passes

@github-actions
Copy link

github-actions bot commented Mar 8, 2026

✅ StructLint Validation

Metric Count
Checks passed 76
Violations 0

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Core Changes

  • Introduces a new GitHub Actions workflow (pr-title.yml) to validate Pull Request titles.
  • Enforces Conventional Commits specification for PR titles (type, optional scope, optional breaking change indicator, and description).
  • The workflow runs on pull_request events (opened, edited, synchronize, reopened) targeting the main branch.

Verdict

Approve: This is a beneficial addition that helps maintain consistent commit history and adheres to Conventional Commits standards, which can be useful for automated changelog generation and versioning. The implementation is straightforward and correct.


Code review performed by GEMINI - gemini-2.5-flash.

@machado144 machado144 force-pushed the chore/add-pr-title-validation branch from af7b144 to ff8a542 Compare March 8, 2026 14:33
@machado144 machado144 changed the title ci: add PR title conventional commit validation ci: consolidate PR workflows into single pr.yml Mar 8, 2026
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Core Changes

  • Consolidates PR-related GitHub Actions workflows into a single pr.yml file.
  • Introduces a new job to validate PR titles against the Conventional Commits specification, posting a comment and failing the check if the title is invalid.
  • Integrates existing AI code review (reviewforge) and structural linting (structlint) into the consolidated PR workflow.

Verdict

Approve - The changes correctly consolidate workflows and add valuable PR title validation without introducing any critical issues.


Code review performed by GEMINI - gemini-2.5-flash.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Core Changes

  • Consolidates PR-related GitHub Actions workflows into a single pr.yml file.
  • Introduces a new job to validate PR titles against the Conventional Commits specification, posting a comment and failing the check if the title is invalid.
  • Integrates existing AI code review (reviewforge) and structural linting (structlint) into the consolidated PR workflow.

Verdict

Approve - The changes correctly consolidate workflows and add valuable PR title validation without introducing any critical issues.


Code review performed by GEMINI - gemini-2.5-flash.

@machado144 machado144 force-pushed the chore/add-pr-title-validation branch from ff8a542 to cd83ddf Compare March 8, 2026 14:35
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Core Changes

  • Consolidates Self-Review.yml and Validate Structure.yml into a single pr.yml workflow.
  • Introduces a new job in pr.yml to validate Pull Request titles against the Conventional Commits specification, including support for breaking change indicator !.
  • Updates the lefthook.yml configuration to align its commit message validation regex with the Conventional Commits specification, specifically adding support for the ! breaking change marker.

Verdict

Approve: The changes correctly consolidate workflows and enhance commit/PR title linting to support the full Conventional Commits specification. No critical issues or regressions were found.


Code review performed by GEMINI - gemini-2.5-flash.

@machado144 machado144 force-pushed the chore/add-pr-title-validation branch from cd83ddf to eba21ba Compare March 8, 2026 14:40
@machado144 machado144 changed the title ci: consolidate PR workflows into single pr.yml ci: consolidate PR workflows and integrate releaseforge for auto-versioning Mar 8, 2026
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Core Changes

  • Consolidated PR-related GitHub Actions workflows (Self-Review.yml, Validate Structure.yml) into a single pr.yml.
  • Implemented PR title validation in pr.yml to enforce Conventional Commits format, including support for the breaking change ! indicator.
  • Enhanced the release.yml workflow to support automatic version bumping using AxeForging/releaseforge@main if a manual tag is not provided, making the tag input optional.
  • Updated the lefthook.yml commit message hook regex to align with the Conventional Commits specification, allowing for the breaking change ! indicator.

Verdict

Approve: The changes are well-implemented, improve CI/CD automation, and correctly enforce Conventional Commits standards without introducing any critical issues.


Code review performed by GEMINI - gemini-2.5-flash.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Core Changes

  • Consolidated PR-related GitHub Actions workflows (Self-Review.yml, Validate Structure.yml) into a single pr.yml.
  • Implemented PR title validation in pr.yml to enforce Conventional Commits format, including support for the breaking change ! indicator.
  • Enhanced the release.yml workflow to support automatic version bumping using AxeForging/releaseforge@main if a manual tag is not provided, making the tag input optional.
  • Updated the lefthook.yml commit message hook regex to align with the Conventional Commits specification, allowing for the breaking change ! indicator.

Verdict

Approve: The changes are well-implemented, improve CI/CD automation, and correctly enforce Conventional Commits standards without introducing any critical issues.


Code review performed by GEMINI - gemini-2.5-flash.

@machado144 machado144 force-pushed the chore/add-pr-title-validation branch from eba21ba to 36b39ad Compare March 8, 2026 14:45
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Core Changes

  • Consolidates PR-related GitHub Actions workflows into a single pr.yml file, removing Self-Review.yml and Validate Structure.yml.
  • Introduces PR title validation in pr.yml to enforce Conventional Commits format, including support for breaking change indicator !.
  • Overhauls the release process in release.yml to use AxeForging/releaseforge for automated version bumping and release note generation based on Conventional Commits, making the tag input optional.

Concerns

None.


Verdict

Approve: The changes significantly improve CI/CD automation and consistency by consolidating workflows and implementing robust release management practices. The logic for PR title validation and automated versioning appears sound and correctly implemented.


Code review performed by GEMINI - gemini-2.5-flash.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@machado144 machado144 force-pushed the chore/add-pr-title-validation branch from 36b39ad to 91c7d5b Compare March 8, 2026 14:56
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Core Changes

  • Consolidates PR-related GitHub Actions workflows (AI review, structlint validation) into a single pr.yml.
  • Introduces a PR title validation job in pr.yml to enforce Conventional Commits format, including support for the breaking change indicator !.
  • Enhances the release.yml workflow to automate version bumping and release note generation using releaseforge, making the tag input optional.
  • Updates the lefthook.yml commit-msg hook regex to align with the Conventional Commits standard, specifically allowing the ! for breaking changes.

Verdict

Approve: The changes significantly improve CI/CD automation and consistency by centralizing PR checks and automating release processes. The implementation is sound and aligns with best practices for conventional commits and GitHub Actions.


Code review performed by GEMINI - gemini-2.5-flash.

@machado144 machado144 merged commit ce8c663 into main Mar 8, 2026
5 checks passed
@machado144 machado144 deleted the chore/add-pr-title-validation branch March 8, 2026 15:01
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