Skip to content

Conversation

lesteve
Copy link
Member

@lesteve lesteve commented Jun 7, 2024

Followed https://learn.scientific-python.org/development/guides/gha-wheels for wheel workflow.
This will be tested in this PR with the [cd build gh] comment. Actually the "Upload to Anaconda" is skipped but I triggered in on my fork and everything looks fine, see https://github.com/lesteve/scikit-learn/actions/runs/9414784335.

The lint workflow is probably fine because there is a single artifact.

For more details about the v3 -> v4 upgrade: https://github.blog/2024-02-12-get-started-with-v4-of-github-actions-artifacts/#compatibility

Context: Dependabot PR #29203

Copy link

github-actions bot commented Jun 7, 2024

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: db8ef9b. Link to the linter CI: here

@lesteve lesteve added the Quick Review For PRs that are quick to review label Jun 7, 2024
Copy link
Member

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@thomasjpfan thomasjpfan merged commit ea1e8c4 into scikit-learn:main Jun 7, 2024
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Jul 2, 2024
@jeremiedbb jeremiedbb mentioned this pull request Jul 2, 2024
11 tasks
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build / CI Quick Review For PRs that are quick to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants