-
-
Notifications
You must be signed in to change notification settings - Fork 91
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EditorConfig hierarchy not working as expected #987
Comments
The logic for this is currently
I'm thinking there are two possible fixes.
The first option is definitely easier to implement and also a lot easier to understand how it works. It seems like the correct way to handle this. |
Option 1 sounds good to me. |
belav
added a commit
that referenced
this issue
Nov 7, 2023
belav
added a commit
that referenced
this issue
Nov 8, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In the root directory I have
.csharperrc.yaml
which specifies the line length as 120.Also, in the root directory I have
.editorconfig
which doesn't specify the line length.In subdirectory
A
I have an.editorconfig
which doesn't specify the line length either. (Note it also does not haveroot = true
)When I run
dotnet csharpier .
in the root directory, all the files inA
get formatted with line length100
(the default).I expected the line length in the root
.csharpierrc.yaml
to be used, but it only does this when theA/.editorconfig
doesn't exist.I've now added
max_line_length = 120
to the root.editorconfig
and it works, but this was a surprise when upgrading to 0.26.The text was updated successfully, but these errors were encountered: