Repository navigation
Experimental CLI does not support max_line_length from .editorconfig #71
Description
Activity
Interesting 🤔
That is indeed not supported at the moment in the thing we are using for parsing
.editorconfigfiles.Interestingly there's no mention of this property in EditorConfig's own website, but it seems mentioned in their wiki, that's why I didn't know this existed.
It should be an easy fix, thanks for reporting the issue 👍
Reacted by Holger Jerominref https://prettier.io/docs/options#print-width
and other references toeditorconfigat this page.While we are on this, please support
"off"Though there is an open issue about this, prettier/prettier#17514, let's just align with the current behavior of Prettier.
@fisker is there an issue in editorconfig's repo tracking that? 🤔 Is "off" documented in Prettier's docs?
is there an issue in editorconfig's repo tracking that?
As far as I know, there is not.
Is "off" documented in Prettier's docs?
No, it's undocumented, but it's better to stay the way it works. We properly not going to do anything about prettier/prettier#17514.
Environments:
Steps to reproduce:
..\node_modules\.bin\prettier.cmd --ignore-path ../.prettierignore --write .and
..\node_modules\.bin\prettier.cmd --ignore-path ../.prettierignore --experimental-cli --write .produces the same output. Hurray.
But having
max_line_length = 120in.editorconfigis not handled in experimental-cli.