Skip to content

Commit

Permalink
chore: allow long commit message titles if they're dependabot-generat…
Browse files Browse the repository at this point in the history
…ed (#450)

## Description

Dependabot generates commit message titles for the dependency update PRs
it makes and _sometimes_ those titles extend longer than our title-check
length allows (i.e. > 100chars).

This PR updates the commitlint config to allow our title-check to _not_
fail on long dependabot commit message titles.

## Type of change

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [x] Other (security config, docs update, etc)

## Checklist before merging

- [ ] Test, docs, adr added or updated as needed
- [ ] [Contributor Guide
Steps](https://docs.pepr.dev/main/contribute/#submitting-a-pull-request)
followed
  • Loading branch information
btlghrants authored Nov 1, 2024
1 parent 6ba1dc1 commit 1031d88
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion commitlint.config.js
Original file line number Diff line number Diff line change
@@ -1 +1,8 @@
module.exports = { extends: ["@commitlint/config-conventional"] };
module.exports = {
extends: ["@commitlint/config-conventional"],
ignores: [
// prevent header-max-length error on long, dependabot-gen'd commits titles
// https://github.com/dependabot/dependabot-core/issues/2445
message => /^chore: bump .+ from .+ to .+$/m.test(message),
],
};

0 comments on commit 1031d88

Please sign in to comment.