Bela VanderVoort

Create documentation explaining CSharpier formatting decisions

3

### Discussed in https://github.com/belav/csharpier/discussions/778 Originally posted by **andrewrondeau-optirtc** December 27, 2022 Hi, apologies if this is a noob question! Do you have any documentation that **explicitly** explains the style that...

?? causes code churn

1

A suffix of `??` would ideally not affect the indentation of the invocations. ```c# return this.CallMethod_____________________() .CallMethod_____________________() .CallMethod_____________________(); return this.CallMethod_____________________() .CallMethod_____________________() .CallMethod_____________________() ?? string.Empty; // is formatted as return this.CallMethod_____________________()...

With how `#if` directives are formatted CSharpier will end up producing a different result depending on the condition. In other words, these two end up with different results. ```csharp CallMethod(...

``` return identityDbContext.UserRoles .Join( identityDbContext.Users, userRole => userRole.UserId, user => user.Id, (userRole, user) => new { user.UserName, userRole.RoleId } ) .Where(o => roleIds.Contains(o.RoleId)) .Select(o => o.UserName) .ToList() as IList; //...

We have a repo that has ~24,000 files. The `Listing all files tracked by git` group contains a massive output of all of those files. I'm not sure how to...

Handle imports that contain PackageReferences.

3

Within a csproj, include a line like ``` ``` And the content of `AnotherFile.props` ``` ``` Ideally AnotherFile.props would also be included in the conversion. Possibly by including *.props in...

I'm looking at [adding editorconfig support to CSharpier](https://github.com/belav/csharpier/issues/630), but CSharpier currently uses `System.IO.Abstractions.IFileSystem` instead of `System.IO` directly, which will cause some pain with CSharpier's tests. Switching `editorconfig-core-net` to `IFileSystem` was...

The current formatting of nested ternary operators ```c# var languageCode = something.IsNotBlank() ? something : otherThing.IsNotBlank() ? otherThing : thingThing.IsNotBlank() ? thingThing : "enUs"; ``` If that were to change...

closes #1220 I mostly went with how prettier works for overrides. Using a file glob, a user can specify options to be used for a file. One of those options...

Right now csharpier eagerly parses all `.editorconfig` files in the directory being formatted, plus its subdirectories. This was a simpler way to deal with peformance than keeping track of which...