Skip to content

Latest commit

 

History

History
49 lines (35 loc) · 1.7 KB

File metadata and controls

49 lines (35 loc) · 1.7 KB

Rule suppress suggestion - Decision

Meeting

Date for meeting

05-02-2021

ATC Core members in meeting

Rule summary

Value
Title SA1122UseStringEmptyForEmptyStrings
CheckId SA1122
Category Documentation Rules
Link https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1122.md

What is the problem

Issue #40

Decision

We update the root/.editorconfig with severity = error and a doc-link. We update the test/.editorconfig with severity = none and a doc-link.

Reason & arguments

  • It explicitly clarifies our intent.
  • Its easier to spot string.Empty than "".
  • Its easier to search for string.Empty than a literal.
  • Eliminates zero width space issues LINK.
  • Minor performance gain.

But for test projects, it can be relevant to use "" then testing.