Thanks to visit codestin.com
Credit goes to github.com

Skip to content

CI exclude cython 3.2.9 as well - #34584

Merged
lesteve merged 1 commit into
scikit-learn:mainfrom
jeremiedbb:extend-temp-exclude-cython-versions
Jul 27, 2026
Merged

CI exclude cython 3.2.9 as well#34584
lesteve merged 1 commit into
scikit-learn:mainfrom
jeremiedbb:extend-temp-exclude-cython-versions

Conversation

@jeremiedbb

Copy link
Copy Markdown
Member

cython/cython#7846 will be released in 3.2.10 so the current exclusion pattern still makes the lock file updates fail.

I can either include the new lock files here or manually trigger the bot regenerate the lock files after we merge this (I prefer the latter)

cc/ @lesteve

@jeremiedbb jeremiedbb added Build / CI Quick Review For PRs that are quick to review labels Jul 27, 2026

@betatim betatim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for extending the list of versions to skip.

Will you include the updated lockfiles in this PR or make a new one? Either is fine for me

@lesteve

lesteve commented Jul 27, 2026

Copy link
Copy Markdown
Member

Let's merge this and trigger the lock-file update via workflow_dispatch which was the approach proposed by @jeremiedbb IIUC.

@lesteve
lesteve merged commit e5f6076 into scikit-learn:main Jul 27, 2026
52 checks passed
@lesteve

lesteve commented Jul 27, 2026

Copy link
Copy Markdown
Member

For the record, in an ideal world, I think I find the update lock-files in PR more work but cleaner (you make sure in the PR that it actually works). In this case, the feeling is that we understand the problem well enough so that the CI will be green with a somewhat high-enough probability 🤞.

For now the workflow trying to update the main lock-file is still running, see build log.

prady0t pushed a commit to prady0t/scikit-learn that referenced this pull request Sep 2, 2026
@jeremiedbb
jeremiedbb deleted the extend-temp-exclude-cython-versions branch September 3, 2026 15:25
@jeremiedbb jeremiedbb mentioned this pull request Sep 8, 2026
14 tasks
jeremiedbb added a commit to jeremiedbb/scikit-learn that referenced this pull request Sep 8, 2026
jeremiedbb added a commit to jeremiedbb/scikit-learn that referenced this pull request Sep 9, 2026
jeremiedbb added a commit to jeremiedbb/scikit-learn that referenced this pull request Sep 9, 2026
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