From 6ef8100e252838532576c1e9ba820e26c05ef0a9 Mon Sep 17 00:00:00 2001
From: Jelle Raaijmakers <jelle@gmta.nl>
Date: Wed, 18 Jan 2023 14:33:45 +0100
Subject: [PATCH] CI: Ignore dependabot in commit linter for PRs

Dependabot cannot be configured to significantly change the way it
formats its commit message, and it currently includes a "Signed-Off-By"
tag which is not allowed by our linter.

This updates our CI commit linter to exclude bots from the checks.
---
 .github/workflows/lintcommits.yml | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/lintcommits.yml b/.github/workflows/lintcommits.yml
index 2d0bd16d3e0..ade728c8db7 100644
--- a/.github/workflows/lintcommits.yml
+++ b/.github/workflows/lintcommits.yml
@@ -15,6 +15,9 @@ jobs:
         uses: actions/github-script@v6
         with:
           script: |
+            const excludedBotIds = [
+              49699333,  // dependabot[bot]
+            ];
             const rules = [
               {
                 pattern: /^[^\r]*$/,
@@ -62,7 +65,10 @@ jobs:
             const commits = await github.paginate(opts);
 
             const errors = [];
-            for (const { sha, commit: { message } } of commits) {
+            for (const { sha, commit: { message }, author } of commits) {
+              if (excludedBotIds.includes(author.id)) {
+                continue;
+              }
               const commitErrors = [];
               for (const { pattern, error } of rules) {
                 if (!pattern.test(message)) {