-
Notifications
You must be signed in to change notification settings - Fork 73
test PR for release 0.9 #1040
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
base: release/0.9
Are you sure you want to change the base?
test PR for release 0.9 #1040
Conversation
abffb4a to
6805b52
Compare
|
|
||
| jobs: | ||
| generate-matrix: | ||
| uses: pytorch/test-infra/.github/workflows/generate_binary_build_matrix.yml@main |
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.
Make sure to rely on the release/2.9 branch of test-infra for all these yaml files. You'll also need to update test-infra-ref: main.
|
|
||
| [build-system] | ||
| requires = ["setuptools>=61.0"] | ||
| requires = ["setuptools>=69.0"] |
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.
were the changes above needed? I'm surprise we don't need them in main?
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.
The license-files changes were needed to resolve this error on python 3.14. On review, I am not certain that requires a newer version of setuptools, though.
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.
This is fishy (and sounds like a regression). We established the license-files = ["LICENSE"] way in #789
This is either a bug in setuptools, or we're doing something wrong here.
8669861 to
92b1572
Compare
No description provided.