-
Notifications
You must be signed in to change notification settings - Fork 6
One python version per container #22
One python version per container #22
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome job. it's coming together.
left a few comments/thoughts.
.github/workflows/prs.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you rename this file back to pr.yaml
? That's the convention that most RAPIDS repositories follow. mambaforge-cuda
is the exception:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I also rename merge.yml
to build.yml
like other RAPIDS repos then?
@ajschmidt8 this is ready for a re-review |
Co-authored-by: AJ Schmidt <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some initial requests:
- Move the deletion of the disabled workflows to a separate PR
- Rename workflows to match the workflows used in our other repos (pr/build/test) assuming they follow the same logic (I don't think we need the equivalent of a "test" workflow nightly)
- Let's leave the
ci-wheel
images named something withmanylinux
in them. Perhapsmanylinux_cuda
? I still want that use case to be clear. - Can we separate change to the manylinux containers from changes to the citestwheel images (separate PRs)?
Hmm. I was hoping to keep these as Once that happens, I assumed we'd have two images in that repository: Any thoughts on this approach? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Images look good. Didn't review the rest.
Co-authored-by: AJ Schmidt <[email protected]>
rapidsai/cibuildwheel-imgs#22 allowed for new wheel containers to be built with only one python version per container
No description provided.