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

Make sure we keep the rapids-cmake and cuml cal version in sync #4213

Conversation

robertmaynard
Copy link
Contributor

when we make a new cuml version, we need to also bump the rapids-cmake version at the
same time. Otherwise we will get the previous releases dependencies by mistake.

when we make a new cuml version, we need to also bump the rapids-cmake version at the
same time. Otherwise we will get the previous releases dependencies by mistake.
@robertmaynard robertmaynard added bug Something isn't working non-breaking Non-breaking change CMake labels Sep 20, 2021
@robertmaynard robertmaynard requested a review from a team as a code owner September 20, 2021 17:25
@robertmaynard robertmaynard added this to PR-WIP in v21.10 Release via automation Sep 20, 2021
@github-actions github-actions bot added gpuCI gpuCI issue and removed CMake labels Sep 20, 2021
v21.10 Release automation moved this from PR-WIP to PR-Reviewer approved Sep 20, 2021
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (branch-21.10@f415f92). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.10    #4213   +/-   ##
===============================================
  Coverage                ?   85.98%           
===============================================
  Files                   ?      231           
  Lines                   ?    18511           
  Branches                ?        0           
===============================================
  Hits                    ?    15917           
  Misses                  ?     2594           
  Partials                ?        0           
Flag Coverage Δ
dask 46.71% <0.00%> (?)
non-dask 78.59% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f415f92...bfd5fdb. Read the comment docs.

@dantegd
Copy link
Member

dantegd commented Sep 20, 2021

@gpucibot merge

@robertmaynard
Copy link
Contributor Author

rerun tests

@rapids-bot rapids-bot bot merged commit 9f9e16f into rapidsai:branch-21.10 Sep 21, 2021
v21.10 Release automation moved this from PR-Reviewer approved to Done Sep 21, 2021
@robertmaynard robertmaynard deleted the keep_rapids-cmake-version-in-sync branch September 22, 2021 13:24
vimarsh6739 pushed a commit to vimarsh6739/cuml that referenced this pull request Oct 9, 2023
…dsai#4213)

when we make a new cuml version, we need to also bump the rapids-cmake version at the
same time. Otherwise we will get the previous releases dependencies by mistake.

Authors:
  - Robert Maynard (https://github.com/robertmaynard)

Approvers:
  - AJ Schmidt (https://github.com/ajschmidt8)
  - Dante Gama Dessavre (https://github.com/dantegd)

URL: rapidsai#4213
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working gpuCI gpuCI issue non-breaking Non-breaking change
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants