Windows line endings in config breaks CLI

[crutchcorn]

Expected Behavior

If you have windows line endings in your config file, things should run as normal

Current Behavior

If you run the recommended echo command on Windows, it will break things as it will output Windows Line Endings.

Affected packages

• cli
• core
• prompt
• config-angular

Possible Solution

Add support for windows line endings in CLI

Steps to Reproduce (for bugs)

1. Open Windows machine
2. Run echo "module.exports = {extends: ['@commitlint/config-angular']};" > commitlint.config.js
3. Run commitlint

Context

Trying to setup commitlint in my project on Windows

Your Environment

Executable	Version
commitlint --version	5.6.0
git --version	git version 2.16.0.windows.2
node --version	v8.9.1

[crutchcorn]

Also seen in #231

[marionebl]

I don't have a win32 machine and can't reproduce this on macOS:
floatdrop/require-from-string#17 (comment)

Can you please add the full stack trace to the report? Also, what terminal emulator are you using to create commitlint.config.js?

[crutchcorn]

How can I get the full stack trace?

I'm using CMDer (conemu)

[marionebl]

I meant the complete console output as you see it in the terminal window where the error is thrown.

[marionebl]

Can you upload your commitlint.config.js as is? Maybe I can reproduce using your file.

[crutchcorn]

commitlint.config.js.zip

C:\Users\CorbinCrutchley\git\test [master +3 ~0 -0 !]
λ  echo 'foo: bar' | node .\node_modules\commitlint\cli.js
C:\Users\CorbinCrutchley\git\test\commitlint.config.js:1
(function (exports, require, module, __filename, __dirname) { ��m
                                                              ^

SyntaxError: Invalid or unexpected token
    at createScript (vm.js:80:10)
    at Object.runInThisContext (vm.js:139:10)
    at Module._compile (module.js:599:28)
    at requireFromString (C:\Users\CorbinCrutchley\git\test\node_modules\require-from-string\index.js:28:4)
    at parseJsFile (C:\Users\CorbinCrutchley\git\test\node_modules\cosmiconfig\dist\loadJs.js:15:15)
    at <anonymous>

[marionebl]

I fumbled around with the file (thanks for providing it!) and my tests indicate this is not caused by \r\n line endings.

It is the encoding, which is UTF-16 LE. cosmiconfig assumes config files are encoded in UTF-8 and consequently feeds a string with invalid characters to require-from-string.

There is nothing commitlint can do to support UTF-16 LE encoded files. I guess the cosmiconfig won't add support for other encodings, so your best bet will be to ensure config files are UTF-8 encoded.

[crutchcorn]

I understand limitations in programs and such, but if I might offer a suggestion - it might be nice to have this documented for others in the future who may run into the same problem. I thought it was a windows issue because dos2unix fixed the problem

[ChiuMungZitAlexander]

The default config of encoding of VSCode in my machine is UTF-16 LE. I met the same problem when executing scripts in the VSCode terminal (powershell) as @crutchcorn did. It was just solved after I changed the encoding to UTF-8.

[indraraj26]

By Default in VS code i have UTF-8 still it gives me error

All files pass linting.
husky > commit-msg (node v8.14.0)
D:\ng\commitlint.config.js:1
(function (exports, require, module, __filename, __dirname) { ��m
                                                              ^

SyntaxError: Invalid or unexpected token
    at createScript (vm.js:80:10)
    at Object.runInThisContext (vm.js:139:10)
    at Module._compile (module.js:617:28)
    at requireFromString (D:\ng-sma\node_modules\require-from-string\index.js:28:4)
    at parseJsFile (D:\ng-sma\node_modules\cosmiconfig\dist\loadJs.js:15:15)
    at <anonymous>
husky > commit-msg hook failed (add --no-verify to bypass)

commitlint.config.js

module.exports = {extends: ['@commitlint/config-conventional']}

[YupongUU]

I got the same bug. Any one else solve this bug?

[ooliver1]

for anyone still coming back after this long, i had the issue and i fixed it after 30m of head smashing

Get-Content commitlint.config.js | Set-Content -Encoding utf8 commitlint.config-utf8.js

for windows then delete original and rename new (so it doesnt say used by another process)