Skip to content

tools: fix scripts - use run-time-error instead of run-time-error-cjs #1586

tools: fix scripts - use run-time-error instead of run-time-error-cjs

tools: fix scripts - use run-time-error instead of run-time-error-cjs #1586

Status Failure
Total duration 26s
Artifacts

commitlint-pull-request.yaml

on: pull_request
commitlint
19s
commitlint
Fit to window
Zoom out
Zoom in

Annotations

1 error and 1 warning
commitlint
You have commit messages with errors ⧗ input: tools: fix scripts - use run-time-error instead of run-time-error-cjs 1. The `-cjs` version of the package was put into place because in the production packages we needed it to be able to import the dependency despite us not using ESM just yet. 2. We did a blanket search and replace at the time which seemed like a good idea but turns out it wasn't because there are some tooling scripts (the ones in this diff) that are already using ESM and for those the -cjs flavored package started breaking the code. 3. So this change is just a partial/selective/surgical revert of the earlier project-wide migratin from run-time-error to run-time-error-cjs. Signed-off-by: Peter Somogyvari <[email protected]> ✖ type must be one of [build, chore, ci, docs, feat, fix, perf, refactor, revert, style, test] [type-enum] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
commitlint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/[email protected]. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.