Skip to content

Update README.md #66

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

Closed
wants to merge 1 commit into from
Closed

Update README.md #66

wants to merge 1 commit into from

Conversation

dave3d
Copy link
Member

@dave3d dave3d commented Jul 11, 2025

No description provided.

@dave3d dave3d requested a review from zivy July 11, 2025 20:25
We use [PySpellChecker](https://github.com/barrust/pyspellchecker) for our
underlying dictionary. Previously we used different package that required an
underlying C library, which was not available on all platforms.
PySpellChecker is pure Python and works on all platforms.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this describes the relevant changes. Primary change is in underlying spellchecker, with a secondary change in the default dictionary. Proposed language:

We use PySpellChecker as the underlying spellchecker and its associated default dictionary. Previously we used the pyenchant package and its default dictionary. The pyenchant package requires an underlying C library, which is not available on all platforms. PySpellChecker is a pure Python package and works on all platforms with no additional dependencies.

@dave3d dave3d closed this Jul 14, 2025
@dave3d dave3d deleted the UpdateREADME branch July 14, 2025 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants