"editorconfig" configuration prints an "Ignored unknown option" warning

[Shayan-To]

Adding { "editorconfig": true } to the configuration file, although it works, but prints [warn] Ignored unknown option { editorconfig: true }. (multiple times).

Environments:

• Prettier Version: 2.7.1
• Running Prettier via: CLI
• Runtime: Node.js v16.14.2
• Operating System: Ubuntu 20.04.4
• Prettier plugins (if any): --

Steps to reproduce:
Add { "editorconfig": true } to .prettierrc.json and run prettier.

Expected behavior:
No warn message is printed.

Actual behavior:
This message is printed (multiple times): [warn] Ignored unknown option { editorconfig: true }.

[thorn0]

That's an option of Prettier CLI, not something specified in a config file.

[Shayan-To]

@thorn0 So I think it should be removed from the json schema (it is the fifth property).

[thorn0]

Indeed, it shouldn't be there unless I'm missing something. Need to investigate. Thanks for noticing this.

[danielpza]

That's an option of Prettier CLI, not something specified in a config file.

Is there any reason it cannot be specified in the config file? It actually recognizes the option if placed there.

The docs makes no mention of this either https://prettier.io/docs/en/configuration.html#editorconfig

[kotx]

Why isn't this an option in .prettierrc? It would make sense that CLI flags roughly match config file options.

[bschaeublin]

This problem has been bothering me for some time. If I set editorconfig: true as specified in the config scheme, the editorconfig settings are respected when saved in webstorm (with prettier enabled).

But the ci check npx prettier --check . gives a warning about this "unknown" option.
It would be nice if this option was still supported in the prettier-options file, so that IDE integrations don't have to be configured separately.

[danielpza]

Can a prettier maintainer please address this issue? It doesn't make sense to be a cli option but not in the config file. Is the editorconfig option going to be supported in the .prettierrc file or not?

Are you open to PR to remove the warning and update the docs?

[rotu]

I'm finding that "editorconfig" is not respected in .prettierrc.json. It's true by default, and if I set to false in the config file, prettier still pulls settings from .editorconfig.