new python wrapper#75
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #75 +/- ##
==========================================
+ Coverage 58.07% 58.41% +0.33%
==========================================
Files 31 32 +1
Lines 2359 2366 +7
Branches 290 290
==========================================
+ Hits 1370 1382 +12
+ Misses 982 977 -5
Partials 7 7 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds a new Python wrapper around libdedx using ctypes, supported by a new shared-library build target and CI coverage for the Python smoke tests.
Changes:
- Introduce a pure-Python
ctypesbinding (python/libdedx.py) plus type stubs and pytest smoke tests. - Update CMake to build/install both static and shared
dedxlibraries (shared needed for dynamic consumers like Python). - Add/extend CI and dev-environment configuration for running Python tests, and adjust install smoke test runtime environment.
Reviewed changes
Copilot reviewed 15 out of 16 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
tests/install_smoke_test.cmake |
Runs the installed consumer with an environment override for runtime library lookup. |
python/tests/test_libdedx.py |
Adds basic smoke tests for the Python binding. |
python/tests/conftest.py |
Auto-detects the built shared library path for local/CI test runs. |
python/test.py |
Removes the old ad-hoc Python test script. |
python/pyproject.toml |
Adds packaging metadata for installing the Python binding via setuptools. |
python/libdedx.pyi |
Adds typing stubs for the Python API. |
python/libdedx.py |
Implements the new ctypes wrapper API around C wrapper functions. |
python/libdedx.c |
Removes the old CPython extension implementation. |
python/README |
Removes the old Python binding README/instructions. |
python/CMakeLists.txt |
Removes old CMake-based extension build logic. |
libdedx/CMakeLists.txt |
Builds a shared library target alongside the static library (via an object library). |
.vscode/settings.json |
Adds Python-specific VS Code settings for analysis/testing. |
.vscode/extensions.json |
Updates recommended VS Code extensions (adds Python). |
.gitignore |
Ignores Python venv/cache/build artifacts. |
.github/workflows/python.yml |
Adds a dedicated Python workflow that builds and runs Python tests. |
.github/workflows/ci.yml |
Adds a Python test job to the main CI workflow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 17 out of 18 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…n and package configuration
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 17 out of 19 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.