Róbert Papp

Ask yourself if there's a possibility for the case you're flagging. Is it possible to write code that passes/fails the rule? And are both of them valid engineering choices? (i.e....

I can confirm that the fix did not fix the reported problem: https://github.com/TWiStErRob/net.twisterrob.cinema/pull/173/files 

@BraisGabin can you please reopen?

> Seems like a good idea to test this on a windows rig. I have Windows too, what steps would I need to do? > It also means that the...

Oh, there's another two: ``` LibraryCodeMustSpecifyReturnType: active: true excludes: ['**'] LibraryEntitiesShouldNotBePublic: active: true excludes: ['**'] ``` Starting to think this is intentional, just non-intuitive.

I'm configuring one of them to be active, but it looks very strange: ``` style: LibraryCodeMustSpecifyReturnType: excludes: [ ] ```

I tried to read into those, but it's still unclear why it is set up this way. Won't `active: false` achieve the same thing? cc @BraisGabin or is this maybe...

Sounds like a good idea. I think I can see what's up now. Is there action to take in this issue? Convert this a doc improvement or that "extract" task?...

Glad you agree. Sounds like `ConsistentWhenBraces` would be a good add, but I would keep `OptionalWhenBraces` too, because lot of people naturally like to golf their code preferring concise over...

I would like to come back to this, this rule is very inconsistent with `MandatoryBracesIfStatements`, for a trivial if-else, we can require braces, but for very complex `when` the only...