Skip to content

Macos ci #1339

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

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Macos ci #1339

wants to merge 8 commits into from

Conversation

yedayak
Copy link
Collaborator

@yedayak yedayak commented Feb 24, 2025

See #726

scop and others added 6 commits February 24, 2025 22:28
This is an interim hack to remove things from CI config unrelated to
beating the test suite into shape on macOS, for a bit faster turnaround.
To be cleaned up and properly integrated to check.yaml before merge.
brew/python doesn't like installing python packages globally:
```
error: externally-managed-environment
× This environment is externally managed
```
@yedayak
Copy link
Collaborator Author

yedayak commented Feb 24, 2025

The CI run failed because for some reason it didn't manage to delete the temp directory it creates in test_unit_load.py using shutil.rmtree in prepare_fixture_dir

@scop
Copy link
Owner

scop commented Apr 18, 2025

Thanks for working on this! I think I've seen the temp dir removal issue myself as well. Would be great if we could figure out a workaround. If we can't, we should allow it to fail only on macOS.

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