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] Minor cpp improvement #48734

Merged

Conversation

dentiny
Copy link
Contributor

@dentiny dentiny commented Nov 13, 2024

Some C++ improvements to our codebase.

Signed-off-by: dentiny <dentinyhao@gmail.com>
@dentiny dentiny requested review from jjyao and rynewang November 13, 2024 23:10
@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>
Signed-off-by: dentiny <dentinyhao@gmail.com>
Signed-off-by: dentiny <dentinyhao@gmail.com>
Signed-off-by: dentiny <dentinyhao@gmail.com>
Signed-off-by: dentiny <dentinyhao@gmail.com>
Signed-off-by: dentiny <dentinyhao@gmail.com>
@dentiny dentiny requested a review from rynewang November 14, 2024 01:41
Signed-off-by: dentiny <dentinyhao@gmail.com>
Signed-off-by: dentiny <dentinyhao@gmail.com>
@dentiny dentiny force-pushed the hjiang/minor-cpp-improvement-nov-13 branch from 7e7de7c to fde4299 Compare November 14, 2024 03:19
Signed-off-by: dentiny <dentinyhao@gmail.com>
@dentiny dentiny requested a review from rynewang November 15, 2024 00:01
@rynewang rynewang enabled auto-merge (squash) November 15, 2024 00:36
@rynewang rynewang merged commit d05d91e into ray-project:master Nov 15, 2024
6 checks passed
dentiny added a commit to dentiny/ray that referenced this pull request Dec 7, 2024
Some C++ improvements to our codebase.

---------

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