Cannot finish file scan in `eslint --init` when file contains invalid ESLint config comment

What version of ESLint are you using? 2.8.0 What parser (default, Babel-ESLint, etc.) are you using? default I hope Please show your full configuration: There is no configuration yet What did you do? Please include the actual source code causing the issue. eslint --init and I choose to scan my files to determine the best configuration one of files contains invalid comment /*eslint max-statements[2,60], max-depth:[2,4] */ What did you expect to happen? successful finish What actually happened? Please include the actual, raw output from ESLint.

(mention.js:31:13) Failed to parse JSON from ' max-statements[2,60], "max-depth":[2,4]': Unexpected token m
Error: (mention.js:31:13) Failed to parse JSON from ' max-statements[2,60], "max-depth":[2,4]': Unexpected token m
    at getSourceCodeOfFile (C:\Users\khomyackov\AppData\Roaming\npm\node_modules\eslint\lib\util\source-code-util.js:43:15)    at C:\Users\khomyackov\AppData\Roaming\npm\node_modules\eslint\lib\util\source-code-util.js:99:26
    at Array.forEach (native)
    at getSourceCodeOfFiles (C:\Users\khomyackov\AppData\Roaming\npm\node_modules\eslint\lib\util\source-code-util.js:98:15)    at configureRules (C:\Users\khomyackov\AppData\Roaming\npm\node_modules\eslint\lib\config\config-initializer.js:128:23)
    at processAnswers (C:\Users\khomyackov\AppData\Roaming\npm\node_modules\eslint\lib\config\config-initializer.js:250:18)
    at null.completed (C:\Users\khomyackov\AppData\Roaming\npm\node_modules\eslint\lib\config\config-initializer.js:404:30)
    at PromptUI.onCompletion (C:\Users\khomyackov\AppData\Roaming\npm\node_modules\eslint\node_modules\inquirer\lib\ui\prompt.js:57:10)
    at AnonymousObserver.Rx.AnonymousObserver.AnonymousObserver.completed (C:\Users\khomyackov\AppData\Roaming\npm\node_modules\eslint\node_modules\inquirer\node_modules\rx-lite\rx.lite.js:1550:12)
    at AnonymousObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (C:\Users\khomyackov\AppData\Roaming\npm\node_modules\eslint\node_modules\inquirer\node_modules\rx-lite\rx.lite.js:1489:14)

[nzakas]

Fair enough. Then perhaps we can make config comment parsing failure a reported error instead of a thrown error as @victor-homyakov suggested? That would be a change to the core rather than an autoconfig change.

[kaicataldo]

Closing this because we haven’t made any progress in months. Thanks for contributing!