Skip to content

ci: use newer cuda image and cmake 3.21#301

Merged
bd4 merged 1 commit intowdmapp:mainfrom
bd4:pr/ci-cmake321
Mar 3, 2026
Merged

ci: use newer cuda image and cmake 3.21#301
bd4 merged 1 commit intowdmapp:mainfrom
bd4:pr/ci-cmake321

Conversation

@bd4
Copy link
Contributor

@bd4 bd4 commented Mar 2, 2026

No description provided.

@td-mpcdf
Copy link
Contributor

td-mpcdf commented Mar 2, 2026

I think it is not the base image that makes a problem, but the cmake that is build inside the build_cuda action. There it pulls 3.18, should be upgraded to perhaps 3.31.11 (last one of the major 3 versions) or even to a more recent cmake of major 4.

@bd4 bd4 force-pushed the pr/ci-cmake321 branch from e3e6880 to 019df84 Compare March 2, 2026 17:34
@bd4 bd4 changed the title ci: use newer cuda image for cmake >= 3.21 ci: use newer cuda image and cmake 3.21 Mar 2, 2026
@bd4
Copy link
Contributor Author

bd4 commented Mar 2, 2026

@td-mpcdf I think the permissions allow anyone to do a review?

@td-mpcdf
Copy link
Contributor

td-mpcdf commented Mar 3, 2026

I approve the PR, but I do not know if I can click somewhere. Perhaps not as I do not have write access.

@bd4 bd4 merged commit e3f8670 into wdmapp:main Mar 3, 2026
20 checks passed
@bd4 bd4 deleted the pr/ci-cmake321 branch March 3, 2026 14:43
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.

4 participants