Skip to content

Conversation

vtavana
Copy link
Collaborator

@vtavana vtavana commented Apr 17, 2025

add python matrix in build with clang to cover python 3.10, 3.11, 3.12

@vtavana vtavana self-assigned this Apr 17, 2025
@vtavana vtavana changed the title add python matrix to build with clang add python matrix in build with clang Apr 17, 2025
@vtavana vtavana force-pushed the extend-clang branch 3 times, most recently from 4ade332 to 04a37d4 Compare April 17, 2025 05:03
@vtavana vtavana marked this pull request as ready for review April 17, 2025 05:39
@Copilot Copilot AI review requested due to automatic review settings April 17, 2025 05:39
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

.github/workflows/build-with-clang.yml:55

  • The removal of 'numpy' and 'pytest-cov' from the dependencies may lead to build or test failures if these packages are required. Please verify that these removals are intentional.
pip install cython setuptools pytest

Copy link
Collaborator

@antonwolfy antonwolfy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you @vtavana

@vtavana vtavana merged commit fe20347 into master Apr 17, 2025
19 checks passed
@vtavana vtavana deleted the extend-clang branch April 17, 2025 16:25
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