Skip to content

ci(commits): Adopt the shared commit-message validation action - #103

Merged
nfebe merged 1 commit into
devfrom
ci/consistent-commit-validation
Jul 14, 2026
Merged

ci(commits): Adopt the shared commit-message validation action#103
nfebe merged 1 commit into
devfrom
ci/consistent-commit-validation

Conversation

@nfebe

@nfebe nfebe commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Swap the inline commitlint config for the shared whilesmart/workflows/commits@main action. webui already validated on dev; this moves the rules into the shared action (which additionally allows perf) so they live in one place instead of a copy per repository.

Replace the inline commitlint config with the shared action so the commit
rules live in one place across repos instead of a copy per repository.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying trakli-dev with  Cloudflare Pages  Cloudflare Pages

Latest commit: 14ddda6
Status: ✅  Deploy successful!
Preview URL: https://faff760b.trakli-dev.pages.dev
Branch Preview URL: https://ci-consistent-commit-validat.trakli-dev.pages.dev

View logs

@sourceant

sourceant Bot commented Jul 14, 2026

Copy link
Copy Markdown

Code Review Summary

This pull request refactors the commit message validation workflow to use a shared organizational action instead of a local, hardcoded implementation.

🚀 Key Improvements

  • Centralized commit message rules in whilesmart/workflows/commits.
  • Reduced maintenance overhead by removing duplicate configuration.
  • Faster CI execution by avoiding local package installation steps.

💡 Minor Suggestions

  • Consider pinning the shared action to a specific tag (e.g., @v1) instead of @main to prevent unexpected CI breaks.

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review complete. No specific code suggestions were generated. See the overview comment for a summary.

@nfebe
nfebe merged commit f162b53 into dev Jul 14, 2026
6 of 7 checks passed
@nfebe
nfebe deleted the ci/consistent-commit-validation branch July 14, 2026 10:41
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