Skip to content

fix(pr-checks): allow pascal-case and camel-case in commit messages#353

Merged
clouddrove-ci merged 1 commit intomasterfrom
fix/pr-checks-allow-pascal-camel-case
Mar 21, 2026
Merged

fix(pr-checks): allow pascal-case and camel-case in commit messages#353
clouddrove-ci merged 1 commit intomasterfrom
fix/pr-checks-allow-pascal-camel-case

Conversation

@hahirwar-cd
Copy link
Copy Markdown
Contributor

@hahirwar-cd hahirwar-cd commented Mar 21, 2026

Description

Relax commit message casing rules in the PR checks workflow to allow more flexible subject formatting.

  • Disabled subject-case rule in commitlint
  • Disabled type-case rule to allow uppercase variants (e.g., Feat, Fix)
  • Updated type-enum to support both lowercase and uppercase commit types
  • No changes to existing commit validation workflow structure

Type of Change

  • 🐛 Bug fix
  • ✨ New workflow
  • 📝 Documentation update
  • 🔧 Workflow enhancement
  • 🎨 Code style/formatting
  • ♻️ Refactoring
  • ⚡ Performance improvement
  • 🔒 Security improvement

Workflow Category

  • Terraform (tf-*)
  • CloudFormation (cf-*)
  • Docker (docker-*)
  • Helm (helm-*)
  • PR Automation (pr-*)
  • Security (security-*)
  • Release (release-*)
  • Notification (notify-*)
  • AWS-specific (aws-*)
  • GCP-specific (gcp-*)
  • YAML Lint (yl-*)
  • Other

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Testing

  • Commit messages
    image

  • This PR fixes following issues in pr-checks

    ⧗   input: feat: Check CapiTal Letters in CommmiT messAge
    ✖   subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case]
    
    ✖   found 1 problems, 0 warnings
    ⧗   input: Feat: fail commit message
    ✖   type must be lower-case [type-case]
  • PR Checks

image

Related Issues

N/A

Closes

N/A

Additional Notes

N/A

@hahirwar-cd hahirwar-cd requested a review from dverma-cd March 21, 2026 15:48
@hahirwar-cd hahirwar-cd added the enhancement New feature or request label Mar 21, 2026
@hahirwar-cd hahirwar-cd requested a review from maharshi-cd March 21, 2026 15:49
@hahirwar-cd hahirwar-cd self-assigned this Mar 21, 2026
@hahirwar-cd hahirwar-cd requested a review from anket-cd March 21, 2026 15:50
@clouddrove-ci clouddrove-ci merged commit 61a4d25 into master Mar 21, 2026
28 checks passed
@clouddrove-ci clouddrove-ci deleted the fix/pr-checks-allow-pascal-camel-case branch March 21, 2026 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants