From 1031d88a6377666cdd6a11088a75889cd23ec079 Mon Sep 17 00:00:00 2001 From: Barrett <81570928+btlghrants@users.noreply.github.com> Date: Fri, 1 Nov 2024 08:56:33 -0500 Subject: [PATCH] chore: allow long commit message titles if they're dependabot-generated (#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 --- commitlint.config.js | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/commitlint.config.js b/commitlint.config.js index 5073c20..1d54fa2 100644 --- a/commitlint.config.js +++ b/commitlint.config.js @@ -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), + ], +};