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

use macos-14 as macos arm build runner. #306

Merged
merged 2 commits into from
May 25, 2024
Merged

use macos-14 as macos arm build runner. #306

merged 2 commits into from
May 25, 2024

Conversation

graphenn
Copy link
Contributor

use macos-14 as macos arm build runner.

Skip build cpython 3.8 wheel because of error.

@brentp
Copy link
Owner

brentp commented May 24, 2024

what's going on with sdist tests?

@graphenn graphenn force-pushed the debug branch 2 times, most recently from 9a05a6e to b74b1cd Compare May 25, 2024 03:08
@graphenn
Copy link
Contributor Author

The sdist job that running pytest tests needs to ignore the cyvcf2 directory within the folder; otherwise, it will prioritize loading the subdirectory as the cyvcf2 module. This was originally achieved using pytest --import-mode=importlib. However, the tests have failed with pytest versions 8.2.0 to 8.2.1, and the reason is unknown. Simply reverting to an earlier version of pytest resolves the issue.

@brentp brentp merged commit dec3fb1 into brentp:main May 25, 2024
12 checks passed
@brentp
Copy link
Owner

brentp commented May 25, 2024

thank you!

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