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

Add fastspell conservative and agressive to the comparison #190

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

marco-c
Copy link
Contributor

@marco-c marco-c commented Nov 13, 2023

Fixes #188

@marco-c
Copy link
Contributor Author

marco-c commented Nov 13, 2023

The PR is marked as draft because Poetry is failing to install the package.

@marco-c
Copy link
Contributor Author

marco-c commented Nov 23, 2023

I managed to find a way to fix the installation error. With Python 3.10, which I'm using, we need to install at least cyhunspell 2.0.3 (which isn't published on PyPI yet).

@pemistahl
Copy link
Owner

Thank you Marco. I will deal with your PR shortly. MyPy is reporting type checking failures. Can you fix them please?

@pemistahl
Copy link
Owner

You always need to run the Black code formatter as well, please. Otherwise, the build fails.

@marco-c
Copy link
Contributor Author

marco-c commented Nov 24, 2023

Yeah, I'm too used to my projects where mypy, black and other linting are run on a commit hook :)

@pemistahl pemistahl merged commit a5c28f5 into pemistahl:main Nov 27, 2023
5 checks passed
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.

Add fastspell to the comparison
2 participants