Skip to content

Conversation

@dave3d
Copy link
Member

@dave3d dave3d commented Mar 6, 2025

No description provided.

@dave3d dave3d force-pushed the PythonBackend branch 2 times, most recently from c2c1655 to 14650cf Compare March 13, 2025 18:45
@dave3d dave3d requested review from blowekamp and zivy March 13, 2025 19:26
@dave3d dave3d requested review from blowekamp and zivy April 3, 2025 18:16
@dave3d dave3d requested a review from blowekamp April 21, 2025 14:17
Copy link
Member

@blowekamp blowekamp left a comment

Choose a reason for hiding this comment

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

Looking pretty good. One minor comment regarding the testing.

@dave3d dave3d force-pushed the PythonBackend branch 2 times, most recently from 1e18d37 to fd0cb06 Compare May 7, 2025 17:10
This is a major re-write of the code.  It now uses pyspellchecker,
a pure Python package, replacing pyenchant, which required the C
enchant library underneath.

Also code has been re-organized to make seperate functions into
smaller, more manageable parts and to adhere to Pylint's coding standards.
@dave3d dave3d requested a review from blowekamp May 9, 2025 19:57
Copy link
Member

@blowekamp blowekamp left a comment

Choose a reason for hiding this comment

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

The changes look good and seem to follow Python conventions

@dave3d dave3d merged commit b458668 into main May 15, 2025
2 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.

4 participants