Consider - forcing braces on loops and if statements.

[danfma]

Hello,

It would be nice if we could enforce the curly brackets for conditions and loops. Probably, a strategy for that would be better, something like:

• always => always use;
• when_multiline => only when there are more than two statements;

What do you think?

[belav]

CSharpier is an opinionated formatter and adding an option goes against that.

I would consider always forcing braces, but I don't know how split developers are in the always braces vs no braces. There are also other tools to force braces, they just don't run on save.

[jods4]

I feel like this is unlikely to get strong consensus in community.

Personally, I like no-brace style for short single-line statements like if (!alive) return;.
I know other people who swear by always-use-braces.

I think it's probably simpler for CSharpier to stick to its "no-syntax change, whitespace formatting-only" general policy on this one.
Braces can be enforced by linters (even C# by itself can be configured through .editorconfig to error on if without braces).

(BTW in the same category: dropping brackets around single-argument, type-less lambdas? Prettier has an option to change (x) => true into x => true. I like this but I feel my comment applies just the same for this one.)

[danfma]

Yeah, I understand csharpier is opinionated and that adding more options will make the code harder to maintain, so that was just an idea.