search

G-Research / fsharp-analyzers

Analyzers for F#
https://g-research.github.io/fsharp-analyzers/
Apache License 2.0
14 stars 1 forks source link

Colons should also be allowed inside log templates #78

Closed nojaf closed 7 months ago

nojaf commented 7 months ago

From the trenches:

logger.LogDebug("Blah {s_thing:l} foo {s_other_thing:l}", someString, someOtherString)

See https://github.com/serilog/serilog/wiki/Formatting-Output

Colons are allowed in this case.

Can you check this please @dawedawe?

Smaug123 commented 7 months ago

There's a chance there's no correct validation possible here and we should just detect grouped braces.