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

Move cmake to the build section. #763

Merged
merged 1 commit into from
Jul 28, 2022

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Jul 28, 2022

As a CLI tool CMake belongs in the build section and shouldn't need to be present in the host requirements.

@vyasr vyasr added 3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 28, 2022
@vyasr vyasr requested a review from a team as a code owner July 28, 2022 00:43
@vyasr vyasr self-assigned this Jul 28, 2022
@cjnolet cjnolet changed the base branch from branch-22.10 to branch-22.08 July 28, 2022 18:49
Copy link
Contributor

@msadang msadang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Version changes and cmake build look good.

CHANGELOG.md Outdated
@@ -1,3 +1,7 @@
# raft 22.10.00 (Date TBD)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like you branched from branch-22.10 but are targeting branch-22.08. Can you fix this?

@cjnolet
Copy link
Member

cjnolet commented Jul 28, 2022

Looks like an intermittent failing ANN ivf flat gtest.

@cjnolet
Copy link
Member

cjnolet commented Jul 28, 2022

rerun tests

@cjnolet
Copy link
Member

cjnolet commented Jul 28, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 642567d into rapidsai:branch-22.08 Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants