fix: build the action with rollup

On v4, we used the default nodejs resolution logic to allow ES modules in dependencies.
This created a breaking change of forcing users of this action
to use the .cjs extension instead of .js in config files.

With this fix, we now bundle the action with rollup to allow ES modules
in dependencies, while keeping the support for .js config files.

With this change, the default config file was returned back to .js instead of
.cjs.

Fixes #194

.dockerignore

@@ -1,5 +1,5 @@
 node_modules
-commitlint.config.cjs
+commitlint.config.js
 action.yml
 .github
 CHANGELOG.md

.eslintrc.json

@@ -18,12 +18,6 @@
     "no-process-exit": "off",
     "node/no-unpublished-require": "off",
     "node/no-unpublished-import": "off",
-    "node/no-unsupported-features/es-syntax": "off",
-    "import/extensions": [
-      "error",
-      {
-        "js": "always"
-      }
-    ]
+    "node/no-unsupported-features/es-syntax": "off"
   }
 }

.gitignore

@@ -59,3 +59,6 @@ typings/
 
 # next.js build output
 .next
+
+# Dist files
+dist

.nvmrc

@@ -1 +1 @@
-16
+16.5.0

.versionrc.json

@@ -5,7 +5,7 @@
     "package-lock.json",
     {
       "filename": "action.yml",
-      "updater": "./.github/tasks/actionYamlUpdater.cjs"
+      "updater": "./.github/tasks/actionYamlUpdater.js"
     }
   ],
   "releaseCommitMessageFormat": "chore(release): publish {{currentTag}} [skip-ci]"

CHANGELOG.md

@@ -18,7 +18,6 @@ All notable changes to this project will be documented in this file. See [standa
 
 * Node.js version used on the action updated from 12 to
 16
-* Config files now need to be renamed from .js to .cjs
 
 ### Features
 

Dockerfile

@@ -1,11 +1,23 @@
-FROM node:16-alpine3.13
+FROM node:16.5.0-alpine3.14 as build
+
+COPY package*.json /
+
+RUN npm ci --ignore-scripts
+
+COPY . .
+
+RUN npm run build
+
+FROM node:16.5.0-alpine3.14
 
 RUN apk --no-cache add git
 
+COPY --from=build dist/run.js /run.js
+
 COPY package*.json /
 
 RUN npm ci --production --ignore-scripts
 
-COPY . .
+COPY entrypoint.sh /entrypoint.sh
 
 ENTRYPOINT ["/entrypoint.sh"]

README.md

@@ -32,7 +32,7 @@ You can supply these inputs to the `wagoid/commitlint-github-action@v3` step.
 
 The path to your commitlint config file.
 
-Default: `commitlint.config.cjs`
+Default: `commitlint.config.js`
 
 If the config file doesn't exist, [config-conventional](https://github.com/conventional-changelog/commitlint/tree/master/%40commitlint/config-conventional) settings will be loaded as a default fallback.
 

action.yml

@@ -3,19 +3,20 @@ description: Lints Pull Request commit messages with commitlint
 author: Wagner Santos
 inputs:
   configFile:
-    description: Commitlint config file. If the file doesn't exist, config-conventional settings will be
+    description:
+      Commitlint config file. If the file doesn't exist, config-conventional settings will be
       loaded as a fallback.
-    default: ./commitlint.config.cjs
+    default: ./commitlint.config.js
     required: false
   firstParent:
     description: >
       When set to true, we follow only the first parent commit when seeing a merge commit. More info
       in git-log docs https://git-scm.com/docs/git-log#Documentation/git-log.txt---first-parent
-    default: "true"
+    default: 'true'
     required: false
   failOnWarnings:
     description: Whether you want to fail on warnings or not
-    default: "false"
+    default: 'false'
     required: false
   helpURL:
     description: Link to a page explaining your commit message convention

babel.config.json

@@ -0,0 +1,6 @@
+{
+  "presets": [["@babel/preset-env"]],
+  "targets": {
+    "node": "16.5.0"
+  }
+}

jest.config.js

@@ -1,6 +1,10 @@
-export default {
+module.exports = {
   // Automatically clear mock calls and instances between every test
+  // preset: 'rollup-jest',
   clearMocks: true,
   testEnvironment: '@commitlint/test-environment',
-  transform: {},
+  transform: {
+    '\\.[jt]sx?$': 'babel-jest',
+  },
+  transformIgnorePatterns: ['node_modules/(?!dargs)'],
 }