-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Install cibuildwheel from requirements file #7581
Conversation
.github/renovate.json
Outdated
}, | ||
{ | ||
"groupName": "requirements-cibw.txt", | ||
"matchFileNames": [".ci/requirements-cibw.txt"] |
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.
Are you sure that renovate won't handle this automatically? https://docs.renovatebot.com/modules/manager/pip_requirements/#file-matching
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.
I'm not sure. Shall we leave this file as is, and check when the next release comes out?
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.
I don't think we'd need to wait until the next release. I think if we merged the PR without this change, then we could see if #6604 started listing the file or not.
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.
Changes to this file dropped.
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.
Yep, now that this is merged, it's there!
So Renovate can update them all at the same time
d323921
to
106f3bc
Compare
Helps #7390.
Switch back to the PyPI-installed cibuildwheel for GitHub Actions, also use it from Travis CI.
The upcoming Windows wheel build can't use the action, so Renovate can update all three at the same time.