-
Notifications
You must be signed in to change notification settings - Fork 930
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
@commitlint/[email protected] throws on load of javascript config missing ts-node #3347
Comments
This was introduced with updating We need the ts-loader for this: https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/load/src/utils/load-config.ts#L30-L36 The Not sure how to deal with this best. Happy for feedback, suggestions, PRs. Easiest way might just be to re-add it as a dep and push out a new version. |
Is there any workaround that we can do for now while the package is not updated yet? |
You should be able to make it work by installing the ts-node package manually. |
Is it possible that commitlint only tries to find the ts-node module when reading a typescript config and otherwise not even looks for it? |
Thanks man. But I preferred to just downgrade the lib to v16.3.0 for now as my project is not a Typescript one. |
Running commitlint with a js config fails for me since today on CI as the load module needs ts-node package, but doesn't find it. I can reproduce it locally.
Expected Behavior
Expected to be able to run commitlint on configs without typescript and without installing additional packages manually.
Current Behavior
Loader fails to run, requires to install ts-node manually.
Affected packages
Steps to Reproduce (for bugs)
The text was updated successfully, but these errors were encountered: