Skip to content
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

[core] [9/N] Skip installation if already #48731

Merged
merged 3 commits into from
Nov 14, 2024

Conversation

dentiny
Copy link
Contributor

@dentiny dentiny commented Nov 13, 2024

Followup and redo for #48668
Addresses comment #48668 (comment)

Signed-off-by: dentiny <dentinyhao@gmail.com>
@dentiny dentiny requested a review from jjyao November 13, 2024 21:34
@dentiny dentiny added the go add ONLY when ready to merge, run all tests label Nov 13, 2024
Signed-off-by: dentiny <dentinyhao@gmail.com>
@jjyao jjyao enabled auto-merge (squash) November 13, 2024 22:17
@github-actions github-actions bot disabled auto-merge November 14, 2024 07:26
@dentiny
Copy link
Contributor Author

dentiny commented Nov 14, 2024

@jjyao Could you please help me merge the PR? I trigger a rerun since the first one fails. Thanks!

@jjyao jjyao merged commit a2d8f63 into ray-project:master Nov 14, 2024
5 checks passed
dentiny added a commit to dentiny/ray that referenced this pull request Dec 7, 2024
Signed-off-by: dentiny <dentinyhao@gmail.com>
Signed-off-by: hjiang <dentinyhao@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go add ONLY when ready to merge, run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants