fix: convert commitlint config to ESM, fix review issues

.github/workflows/main.yml

@@ -18,14 +18,14 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-          node-version: 20
+          node-version: 24
       - run: npm ci
       - run: npm run check
   test:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [20]
+        node-version: [24]
 
     steps:
       - uses: actions/checkout@v4

action.yml

@@ -5,7 +5,7 @@ branding:
   icon: "git-pull-request"
   color: "green"
 runs:
-  using: "node20"
+  using: "node24"
   main: "dist/index.js"
 inputs:
   commitlintRulesPath:

commitlint.config.js

@@ -0,0 +1,3 @@
+export default {
+  extends: ['@commitlint/config-conventional', './commitlint.rules.js']
+}

commitlint.rules.js

@@ -0,0 +1,5 @@
+export default {
+  rules: {
+    'subject-case': [2, 'never', ['start-case', 'pascal-case', 'upper-case']]
+  }
+}

dist/index.js

@@ -43996,7 +43996,7 @@ const getCommitSubject = (commitMessage = '') =>
       checkout:
         'ACTION(commitlintRulesPath): actions/checkout@v2 is required to load your commitlint rules file',
       rules_not_found:
-        'ACTION(commitlintRules ): rules module not found, using default @commitlint/config-conventional lint rules...'
+        'ACTION(commitlintRules): rules module not found, using default @commitlint/config-conventional lint rules...'
     }
   }
 });
@@ -44189,7 +44189,6 @@ async function lintPR() {
   }
 
   const {
-    // eslint-disable-next-line camelcase
     number: pull_number,
     base: {
       user: { login: owner },

package.json

@@ -18,6 +18,9 @@
     "Patrick Heneise <https://patrickheneise.com>"
   ],
   "main": "src/main.js",
+  "engines": {
+    "node": ">=24"
+  },
   "scripts": {
     "check": "opensource-check --path ./",
     "prepare": "ncc build src/main.js -o dist --source-map --license licenses.txt",

src/action-message.js

@@ -16,7 +16,7 @@ export default {
       checkout:
         'ACTION(commitlintRulesPath): actions/checkout@v2 is required to load your commitlint rules file',
       rules_not_found:
-        'ACTION(commitlintRules ): rules module not found, using default @commitlint/config-conventional lint rules...'
+        'ACTION(commitlintRules): rules module not found, using default @commitlint/config-conventional lint rules...'
     }
   }
 }

src/lint-pr.js

@@ -19,7 +19,6 @@ export async function lintPR() {
   }
 
   const {
-    // eslint-disable-next-line camelcase
     number: pull_number,
     base: {
       user: { login: owner },

test/lint-pr.spec.js

@@ -92,10 +92,6 @@ describe('lintPR', () => {
     assert.strictEqual(mockCore.setFailed.mock.callCount(), 0)
   })
 
-  it('fails when contextual pull request is not found', async () => {
-    // TODO: contextual pull request test
-  })
-
   it('skips linting for dependabot PRs', async () => {
     pullsGetResponse = {
       data: { ...prFixture, title: 'bump lodash from 4.0.0 to 4.1.0', user: { login: 'dependabot[bot]' } }