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: Auto-Correct Common Mispellings Allow Users to Ignore Words with Multiple Capitals #1175

Merged
merged 3 commits into from
Sep 27, 2024

Conversation

pjkaufman
Copy link
Collaborator

Fixes #1075
Fixes #1166

There were two requests to allow for auto-correct to ignore capital letters. This is mostly to be used to help make sure acronyms are ignored while still allowing non-acronyms for the same words (i.e. hsa to has vs. HSA) to be properly handled.

Changes Made:

  • Added option to turn off auto-correct for words with multiple capital letters in them
  • Added UTs to check that the rule in question does indeed work

@pjkaufman pjkaufman self-assigned this Sep 27, 2024
@pjkaufman pjkaufman added the markdown General Markdown or Markdown related issue or feature label Sep 27, 2024
@pjkaufman pjkaufman merged commit 646539d into platers:master Sep 27, 2024
1 check passed
@Kainkun
Copy link

Kainkun commented Sep 27, 2024

Thanks!

@pjkaufman
Copy link
Collaborator Author

pjkaufman commented Sep 27, 2024

No problem. Hopefully this helps out with a shortcoming of the rule which was that it was only ever designed to be used with lowercase words/generate lowercase words minus the first letter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
markdown General Markdown or Markdown related issue or feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FR: Auto-correct should ignore words in all caps (aka acronyms) FR: Don't auto-complete acronyms
2 participants