-
Notifications
You must be signed in to change notification settings - Fork 155
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 tool.scikit-build.cmake.version, set scikit-build-core minimum-version #1430
Use tool.scikit-build.cmake.version, set scikit-build-core minimum-version #1430
Conversation
CI is in a bad state here, I don't think it's a result of this PR's changes. We'll need to merge rapidsai/devcontainers#377 to fix the devcontainers builds. And then these others to fix other parts of CI:
I'll check back on this after merging those, before putting this up for review. |
Reported the test failures in #1433. The changes in this PR are ready for review. |
Restarting failed CI jobs now that |
/merge |
This will be blocked until #1434 is complete |
Restarted again now that #1434 is merged. |
Contributes to rapidsai/build-planning#58.
scikit-build-core==0.10.0
was released today (https://github.com/scikit-build/scikit-build-core/releases/tag/v0.10.0), and wheel-building configurations across RAPIDS are incompatible with it.This proposes upgrading to that version and fixing configuration here in a way that:
scikit-build-core
versionminimum-version
) thatscikit-build-core
provides, to reduce the risk of needing to do this again in the future