Unexpected top-level property "lintOnSave".

Version

4.4.1

Reproduction link

https://github.com/davidmoseler/vue-dynamic-table

Environment info

Environment Info:
Killed

Steps to reproduce

Clone following repository and run

npm run lint

What is expected?

No error

What is actually happening?

An error happens although the docs are being followed. According to the docs:

ESLint can be configured via .eslintrc or the eslintConfig field in package.json.

And then it gives the configuration structure only for .eslintrc:

module.exports = {
  lintOnSave: false
}

So I used

"eslintConfig": {
    "lintOnSave": false
}

in package.json, and the output is:

 ERROR  Error: ESLint configuration in package.json is invalid:
        - Unexpected top-level property "lintOnSave".

So, either there is a bug, or the docs could probably have an example of where the lintOnSave property is expected in package.json.

[haoqunjiang]

https://cli.vuejs.org/config/#lintonsave

It's under the section of vue.config.js.

Hi, this is a mistake. As I described, the readme says the option should work inside of package.json, but it does not.

[haoqunjiang]

Which README?

I meant this one:

https://github.com/vuejs/vue-cli/blob/dev/docs/core-plugins/eslint.md

It says:

"ESLint can be configured via .eslintrc or the eslintConfig field in package.json."

And, right after that, it talks about the lintOnSave option. But only now it became clear to me that the lintOnSave option does not work either in .eslintrc or eslintConfig. It is only supposed to work with vue.config.js. It was a misinterpretation from my part. But, yeah, the doc is a little weird in that part. I mean, after the phrase quoted above, it would make sense to at least link to eslintrc configuration instructions. But what comes right after has not much to do with the previous phrase.

[haoqunjiang]

Yeah, that makes sense. PR's welcome.

[jeneser]

@sodatea ping