Define formatting rules

[sebastienros]

Tested with dotnet 8 and dotnet 9 (which adds way more warnings by default, hence these rules).

I tried to disambiguate what contributors hesitate on. For instance method bodies. Now they are fine when fitting a single line. Some things are silent when we do not care much.

The changes in the PR reflect the settings, so that's a good thing to look at to understand the configuration.

[MikeAlhayek]

build is failing

[MikeAlhayek]

I haven't checked. It's possible that client already pass a view model to the call and it may still be there because it was used at some point. Now if you remove the argument the behavior is different as passing something invalid would not fail without it. It's a breaking change to remove the argument.

I don't think this is the case here LayersIndexViewModel does not do any validation. It's there for binding only and since it's not used. It should be removed.

OrchardCore/src/OrchardCore.Modules/OrchardCore.Layers/ViewModels/LayersIndexViewModel.cs

Line 7 in c1f330e

public class LayersIndexViewModel