Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Remove 40 character limit on spell checking words #2175

Merged
merged 10 commits into from
Jan 4, 2022
Merged

Conversation

Jason3S
Copy link
Collaborator

@Jason3S Jason3S commented Jan 4, 2022

Adjust Base64 Detection

The Base64 detection pattern was preventing words longer than 40 characters from being spell checked. It has been turned off by default. To turn it back on add Base64 to ignoreRegExpList in CSpell settings.

A few new patterns were added by default to pick up many of the common cases that Base64 was designed for:

  • Base64SingleLine - a stricter version Base64 detection that only matches single lines.
  • Base64MultiLine - a stricter version Base64 detection that only matches multiple line Base64 strings.
  • Hashes - detects common hash patterns:
    Example: sha1-tdRU3CGZriJWmfNGfloH87lVuv0=
    Supported prefixes:
    • sha1, sha256, sha512, md5, base64, crypt, bcrypt, scrypt, security-token, assertion

Minor Breakage

Because Base64 prevented anything longer than 40 characters from being checked, some misspellings might show up that were previously ignored.

@Jason3S Jason3S merged commit 5769a0e into main Jan 4, 2022
@Jason3S Jason3S deleted the dev-ignore-words branch January 4, 2022 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Does not check words longer than 39 chars Issue in identifying errors in paths
1 participant