[CI] Wheel build, install, and import test to PR_build.yml. #641
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
This commit modifies the `python` job in the `.github/workflows/PR_build.yml` workflow to enhance Python package testing. The changes include: - Modifying the build step to create a Python wheel (`bdist_wheel`) instead of a source distribution (`sdist`). - Adding a step to install the wheel using `pip3`. - Adding a step to perform an import test (`from TheengsDecoder import decodeBLE`) to verify the package is correctly installed and the core module is accessible. - Renaming some steps for clarity. This will help catch packaging and import issues earlier in the PR process.
Contributor
|
Thanks |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description:
This commit modifies the
pythonjob in the.github/workflows/PR_build.ymlworkflow to enhance Python package testing.The changes include:
bdist_wheel) instead of a source distribution (sdist).pip3.from TheengsDecoder import decodeBLE) to verify the package is correctly installed and the core module is accessible.This will help catch packaging and import issues earlier in the PR process.
Checklist: