An existing csharpierrc file always takes priority over editorconfig (#…

…991)

closes #987

Docs/Configuration.md

@@ -7,8 +7,9 @@ CSharpier has support for a configuration file. You can use any of the following
 - A ```.csharpierrc.json``` or ```.csharpierrc.yaml``` file.
 - A ```.editorconfig``` file. See EditorConfig section below.
 
-The configuration file will be resolved starting from the location of the file being formatted, and searching up the file tree until a config file is (or isn’t) found.
-
+The configuration file will be resolved based on the location of the file being formatted.
+- If a `.csharpierrc` file exists somewhere at or above the given file, that will be used.
+- Otherwise if an `.editorconfig` file exists somewhere at or above the given file, that will be used. Respecting editorconfig inheritance.
 ### Configuration Options
 JSON
 ```json

Src/CSharpier.Cli/Options/OptionsProvider.cs

@@ -80,10 +80,7 @@ public PrinterOptions GetPrinterOptionsFor(string filePath)
  return new PrinterOptions();
  }
 
- if (
- (resolvedCSharpierConfig?.DirectoryName.Length ?? int.MinValue)
- >= (resolvedEditorConfig?.DirectoryName.Length ?? int.MinValue)
- )
+ if (resolvedCSharpierConfig is not null)
  {
  return ConfigurationFileOptions.ConvertToPrinterOptions(
  resolvedCSharpierConfig!.CSharpierConfig

Src/CSharpier.Tests/OptionsProviderTests.cs

@@ -446,7 +446,7 @@ public async Task Should_Prefer_CSharpierrc_In_SameFolder()
  }
 
  [Test]
- public async Task Should_Prefer_Closer_EditorConfig()
+ public async Task Should_Not_Prefer_Closer_EditorConfig()
  {
  var context = new TestContext();
  context.WhenAFileExists(
@@ -462,7 +462,7 @@ public async Task Should_Prefer_Closer_EditorConfig()
  "c:/test",
  "c:/test/subfolder/test.cs"
  );
- result.TabWidth.Should().Be(2);
+ result.TabWidth.Should().Be(1);
  }
 
  [Test]