Skip to content

[CI] Wheel build, install, and import test to PR_build.yml. #641

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

Merged
merged 1 commit into from
May 24, 2025

Conversation

1technophile
Copy link
Member

Description:

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.

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

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.
@DigiH
Copy link
Contributor

DigiH commented May 24, 2025

Thanks

@DigiH DigiH merged commit 3f34bd6 into development May 24, 2025
14 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.

2 participants