-
Notifications
You must be signed in to change notification settings - Fork 7.2k
cleanup(ci): don't set default MANYLINUX_VERSION build_arg #60540
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
Conversation
|
Reviews in this chain: |
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.
Code Review
This pull request removes the hardcoded default MANYLINUX_VERSION from several CI configuration files. By relying on a single source of truth from the environment (rayci.env), this change improves maintainability and consistency. The refactoring is applied correctly and consistently across all relevant files. This is a good cleanup.
2691a36 to
92674ad
Compare
Now handled in single source of truth at rayci.env. That envfile should be the only placeholder we update. Topic: cleanup-manylinux Relative: rayci-envfile-27 Signed-off-by: andrew <andrew@anyscale.com>
92674ad to
40ef209
Compare
…ct#60540) Now handled in single source of truth at rayci.env. That envfile should be the only placeholder we update. Topic: cleanup-manylinux Relative: rayci-envfile-27 Signed-off-by: andrew <andrew@anyscale.com> Signed-off-by: andrew <andrew@anyscale.com>
Now handled in single source of truth at rayci.env. That envfile should be the only placeholder we update.
Topic: cleanup-manylinux
Relative: rayci-envfile-27
Signed-off-by: andrew andrew@anyscale.com