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

Maintenance updates #268

Merged
merged 8 commits into from
Feb 6, 2024
Merged

Maintenance updates #268

merged 8 commits into from
Feb 6, 2024

Conversation

aldanor
Copy link
Owner

@aldanor aldanor commented Jan 30, 2024

Some long overdue maintenance updates.

(We should prob release a version soon, are we still stuck with the 1.14 problem?..)

@aldanor aldanor requested a review from mulimoen January 30, 2024 20:51
@aldanor aldanor force-pushed the feature/2024-jan-maintenance branch from 5b3f92c to 90d96e2 Compare January 30, 2024 20:53
Copy link
Collaborator

@mulimoen mulimoen left a comment

Choose a reason for hiding this comment

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

Going to rubber stamp this, would be great to have a new version of these crates out.

I think the wine failure is due to hdf5 built as dev does not build from source with the new resolver version. One could --exclude hdf5-derive for this step, and add another step with cargo test --features hdf5-derive --feature hdf5/static. Update: This fixed compilation, unfortunately running the compile time tests requires some additional magic. I think we can disable this test for wine, as we do for other jobs in CI

The mingw failure I have not been able to replicate locally

@mulimoen mulimoen force-pushed the feature/2024-jan-maintenance branch from f40ac47 to 214dabd Compare January 30, 2024 21:20
@mulimoen mulimoen merged commit 1dff575 into master Feb 6, 2024
40 of 41 checks passed
@mulimoen mulimoen deleted the feature/2024-jan-maintenance branch February 6, 2024 19:50
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