Skip to content

python-bindings: add CI and avoid 'long' integer error#3628

Merged
rhc54 merged 2 commits intoopenpmix:masterfrom
naughtont3:tjn-fixpybind
Jun 7, 2025
Merged

python-bindings: add CI and avoid 'long' integer error#3628
rhc54 merged 2 commits intoopenpmix:masterfrom
naughtont3:tjn-fixpybind

Conversation

@naughtont3
Copy link
Contributor

No description provided.

* OpenPMIx-bindings Python Versions CI checks
   - Builds OpenPMIx Python bindings across different Python versions
   - Uses reusable actions that are driven by python-bindings workflow
       o Actions:   .github/actions/build-python-bindings/action.yaml
       o Workflows: .github/workflows/python-bindings.yaml
   - Setup to get Python per-version results
   - Split actions/workflow to reduce duplicate code, and
     easily add new Python versions (in workflows matrix).
   - Currently only test CPython
   - CI runs limited to pull-requests only (on: pull_request)
   - CI manually trigger on main repo via 'Run Workflow' (on: workflow_dispatch)

Signed-off-by: Thomas Naughton <naughtont@ornl.gov>
* Fix Cython 'long' integer error:
   - Avoids issue for now deprecated use of 'long' integer.
   - As of Python 3.x no longer make this distinction.
   - See also: https://wiki.python.org/moin/PortingExtensionModulesToPy3k#long-int-unification

Signed-off-by: Thomas Naughton <naughtont@ornl.gov>
@rhc54
Copy link
Contributor

rhc54 commented Jun 7, 2025

LGTM - thx!

@rhc54 rhc54 merged commit 684e928 into openpmix:master Jun 7, 2025
25 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