Skip to content
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

Add pytest to workflow #618

Merged
merged 4 commits into from
Feb 24, 2025
Merged

Conversation

cvanelteren
Copy link
Contributor

No description provided.

@molinav
Copy link
Member

molinav commented Feb 24, 2025

The testing needs to be added as part of the new cibuildwheel workflow using CIBW_TEST_COMMAND and related environment variables:
https://cibuildwheel.pypa.io/en/stable/options/#test-command

The commits are also adding refactorings which probably come from a local installation of black.

@cvanelteren
Copy link
Contributor Author

Apologies for the noise. I think cibuild is rather messy for windows, seems to be building now ;-)!

@cvanelteren
Copy link
Contributor Author

I will rebase hold on.

@molinav molinav changed the title add pytest to workflow Add pytest to workflow Feb 24, 2025
@molinav molinav self-assigned this Feb 24, 2025
@molinav molinav merged commit cc0c497 into matplotlib:develop Feb 24, 2025
6 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