Skip to content

fix(renovate): lowercase commitMessageAction to pass PR title validator#52

Merged
williaby merged 2 commits into
mainfrom
fix/renovate-commit-message-casing
May 29, 2026
Merged

fix(renovate): lowercase commitMessageAction to pass PR title validator#52
williaby merged 2 commits into
mainfrom
fix/renovate-commit-message-casing

Conversation

@williaby
Copy link
Copy Markdown
Contributor

@williaby williaby commented May 28, 2026

Summary

  • Add global "commitMessageAction": "update" to renovate.json
  • Lowercase the lockFileMaintenance override that also had "Update"

Why

The PR title validator (amannn/action-semantic-pull-request) uses subjectPattern: ^(?![A-Z]).+$, which rejects subjects beginning with an uppercase letter. Renovate's default commitMessageAction is "Update" (capital U), so every automated Renovate PR fails the Validate PR Title CI check immediately.

This was first observed in PR #46 (chore(deps): Update GitHub Actions), where the subject "Update GitHub Actions" failed because "Update" starts with uppercase "U".

Changes

  • renovate.json: Added "commitMessageAction": "update" at the global level so all future Renovate PRs generate lowercase subjects. Also lowercased the lockFileMaintenance.commitMessageAction override for consistency.

Test plan

  • PR chore(deps): update GitHub Actions #46 title has been manually edited to chore(deps): update GitHub Actions as the immediate fix
  • This config change prevents recurrence for all future Renovate PRs
  • All pre-commit hooks pass on renovate.json

Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated automated dependency update commit message formatting configuration.

Review Change Stack

The PR title validator (amannn/action-semantic-pull-request) uses
subjectPattern: ^(?![A-Z]).+$ which rejects subjects beginning with an
uppercase letter. Renovate's default commitMessageAction is "Update"
(capital U), causing every automated PR to fail the Validate PR Title CI
check.

- Add global "commitMessageAction": "update" so all Renovate PRs
  produce lowercase subjects
- Lower-case the lockFileMaintenance override which also had "Update"

Fixes the recurring CI failure first seen in PR #46.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 28, 2026 21:40
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 86aa9e13-20d2-4f2d-99cf-039c42efdc62

📥 Commits

Reviewing files that changed from the base of the PR and between 0f8b5ce and e78a795.

📒 Files selected for processing (1)
  • renovate.json

Walkthrough

The pull request updates Renovate configuration to normalize the commitMessageAction setting from titlecase "Update" to lowercase "update" in both the lockFileMaintenance block and as a new top-level configuration field.

Changes

Renovate commitMessageAction configuration

Layer / File(s) Summary
commitMessageAction normalization
renovate.json
commitMessageAction is changed from "Update" to "update" within lockFileMaintenance and added as a top-level configuration field set to "update".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A hop through the config, so clean and so neat,
From "Update" to "update"—now the formatting's sweet!
commitMessageAction, lowercase and bright,
Renovate's commits now follow what's right! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: lowercasing the commitMessageAction setting in renovate.json to comply with a PR title validator that rejects uppercase-starting subjects.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/renovate-commit-message-casing

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 28, 2026

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Renovate configuration so Renovate-generated PR titles have a lowercase subject (“update …”), satisfying the repository’s PR title validator regex that rejects subjects starting with an uppercase letter.

Changes:

  • Set global commitMessageAction to "update" so Renovate PR subjects start lowercase by default.
  • Lowercase the lockFileMaintenance.commitMessageAction override for consistency.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 28, 2026

Actionable comments posted: 0

@williaby williaby enabled auto-merge (squash) May 29, 2026 02:05
@sonarqubecloud
Copy link
Copy Markdown

@williaby williaby merged commit 6f93411 into main May 29, 2026
32 checks passed
@williaby williaby deleted the fix/renovate-commit-message-casing branch May 29, 2026 02:08
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