typos icon indicating copy to clipboard operation
typos copied to clipboard

False positive for BRE

Open scop opened this issue 3 years ago • 2 comments

The acronym BRE (short for (POSIX) Basic Regular Expression) gets a false positive hit from typos, suggesting BE or BRIE.

scop avatar Feb 14 '22 20:02 scop

We can't distinguish between straight acronyms and SCREAMING_CASE words, so this comes down to how we do want to treat these as words.

Our options

  • Have people include abbreviations in their dictionary
  • Include this in our dictionary
  • Have a configurable min word length to filter out TLAs

Having a list of acronyms to cover would give me a better sense for the impact of including them.

Short term, I'm going to go ahead with min word lengths.

epage avatar Feb 14 '22 20:02 epage

Bleh, the way concerns are separated, it will take some re-work to add min-length checks

epage avatar Feb 14 '22 21:02 epage

I'm leaning towards BRE being a fairly specific domain word and would recommend people putting it in their dictionary, rather than a more general solution.

We can revisit this depending on what additional input people have.

epage avatar Jan 14 '23 02:01 epage