Open cexbrayat opened 2 years ago
Let's just use the built-in one.
I don't know why codeframe
was chosen as the default formatter.
And the new eslint-formatter-codeframe
is unmaintained (https://github.com/fregante/eslint-formatter-codeframe/issues/2) so it may not be a good replacement.
👍 Which one should we use? https://eslint.org/docs/user-guide/formatters/
ESLint's default stylish
formatter looks good to me.
Sounds good. If that's OK with you, I'll open PRs to:
stylish
as the default formatterYeah, thanks for the help!
In the case of typescript-eslint
, I think you also need a PR to https://github.com/vuejs/eslint-config-typescript/blob/master/package.json
Let's start with that then! Please see https://github.com/vuejs/eslint-config-typescript/pull/34 to update eslint-config-typescript to eslint v8
As a seconde step: PR to switch to stylish formatter https://github.com/vuejs/vue-cli/pull/6748
Third and last step: PR to update new projects to eslint v8 #6749
Note that some plugins are not yet officially compatible with eslint v8, but it looks like it works according to the tests added.
Version
5.0.0-beta.5
Reproduction link
github.com
Environment info
Steps to reproduce
What is expected?
The linting command succeeds
What is actually happening?
The linting command throws:
With the work recently done, eslint v8 nearly works out of the box, except for the codeframe formatter issue, as it has been removed from eslint v8 (see https://eslint.org/docs/8.0.0/user-guide/migrating-to-8.0.0#removed-formatters).
Installing the missing dependency fixes the issue.
Maybe the CLI should install directly? I can work on a PR to bump eslint to v8 and typescript-eslint to v5, and add the missing dependency. Or maybe the default formatter should change to a built-in one? https://eslint.org/docs/user-guide/formatters/