Fix ModuleNotFoundError for url_utils in PyPI distribution and fix search #76
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
vstarlinger
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added explicit package discovery configuration to pyproject.toml to ensure all Python modules are included when building wheels and source distributions. Previously, setuptools was relying on implicit package discovery which could miss certain modules when installing from PyPI, while editable installs worked fine.
The fix adds [tool.setuptools.packages.find] configuration to explicitly discover all packages under src/ matching py_semantic_taxonomy*.
This fixes #74
Also added in this PR the fix for making sure that the search function works for languages. The issue solved here is that clicking on search results didn't persist the language this is solved now by passing it as parameter from the search result.
This fixes #75