Skip to content
This repository was archived by the owner on Jul 28, 2025. It is now read-only.

Updating Clang Version to 11.0.0 - #426

Merged
rapids-bot[bot] merged 8 commits into
rapidsai:branch-21.08from
codereport:clang-11
Jul 6, 2021
Merged

Updating Clang Version to 11.0.0#426
rapids-bot[bot] merged 8 commits into
rapidsai:branch-21.08from
codereport:clang-11

Conversation

@codereport

@codereport codereport commented Jun 26, 2021

Copy link
Copy Markdown
Contributor

Follow up PR to: NVIDIA/cudf#6695. Performing the same changes but for rapidsai/cuspatial

Depends on: rapidsai/integration#304

@codereport
codereport requested review from a team as code owners June 26, 2021 00:37
@github-actions github-actions Bot added conda Related to conda and conda configuration libcuspatial Relates to the cuSpatial C++ library gpuCI Related to Continuous Integration / Automation labels Jun 26, 2021
Comment thread ci/checks/style.sh Outdated
@github-actions github-actions Bot removed the gpuCI Related to Continuous Integration / Automation label Jun 28, 2021

@thomcom thomcom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it expected that the clang version fails in gpuCI/cuspatial/check/style now? Glad to approve when tests pass.

@codereport

Copy link
Copy Markdown
Contributor Author

Is it expected that the clang version fails in gpuCI/cuspatial/check/style now? Glad to approve when tests pass.

Yes, once rapidsai/integration#304 is merged, I will retrigger CI for all the Clang11 format PRs and they will light up green (and once missing labels have been added)

@codereport

Copy link
Copy Markdown
Contributor Author

rerun tests

1 similar comment
@codereport

Copy link
Copy Markdown
Contributor Author

rerun tests

@thomcom thomcom added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 6, 2021
@codereport
codereport requested a review from thomcom July 6, 2021 19:36
@thomcom

thomcom commented Jul 6, 2021

Copy link
Copy Markdown
Contributor

@gpucibot merge

@rapids-bot
rapids-bot Bot merged commit 8c31c2c into rapidsai:branch-21.08 Jul 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

conda Related to conda and conda configuration improvement Improvement / enhancement to an existing function libcuspatial Relates to the cuSpatial C++ library non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants