False positive on "semicolon-separated"

Therzok / MonoDevelop.Analyzers

Set of analyzers that can be used when working on MonoDevelop (and its dependencies)

MIT License

2 stars 2 forks source link

False positive on "semicolon-separated" #28

Open mhutch opened 5 years ago

mhutch commented 5 years ago

Vaclavyzer has a false positive on "semicolon-separated". This is hyphenated, it doesn't show a relationship between two things.

image (1)

I don't think it's possible for this analzyer to work reliably on all cases without a semantic understanding of the sentence. It might be possible to make this work for proper nouns (because they're Title Cased) or dates (because that's a finite set of strings).

Therzok commented 5 years ago

I'll try to fix this and weed out more false positive scenarios.

Therzok commented 4 years ago

We should only be analyzing when a string is passed to a translation service, like GettextCatalog