-
Notifications
You must be signed in to change notification settings - Fork 30
Closed
Description
I'm moving old project from hand-written logger to Serilog. As a temporary solution, some of old usages were replaced with such code:
Although, formally, analyzer is right (String.Empty
is not a constant) and this is actually a bad practice, it is clear that this field will never contain anything other than empty string. Shouldn't String.Empty
case be exceptional? (and, may be, separate code smell rule to warn about empty string template?)
Metadata
Metadata
Assignees
Labels
No labels