Skip to content

[FEA]: Update clang-format to a recent version #1078

Closed

Description

Is this a new feature, an improvement, or a change to existing functionality?

New Feature

How would you describe the priority of this feature request

Medium

Please provide a clear description of problem you would like to solve.

RAPIDS is in the process of updating to a more recent clang-format version. This is already in progress in #1072 .

This will help users of the cuspatial devcontainer because it has a newer clang-format version than is used in CI, and there are often style mismatches as a result.

We may use an earlier version than 16.0.1 due to a bug introduced in that version that results in operator* being formatted as if it is a pointer (a* T{1.0} instead of a * T{1.0}). See https://github.com/rapidsai/cugraph-ops/pull/466#discussion_r1168977855

Describe any alternatives you have considered

No response

Additional context

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

Labels

Type

No type

Projects

  • Status

    Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions