Conversation
ab089ba to
a858b2d
Compare
|
should we have a check for this? |
|
Or a way to plumb it into the normal formatters we run |
Not sure i want to require everyone to have clang-format installed. |
In some way, I'd love CI to just auto-format, re-stage and commit any touched file. Repeating CI runs bc of formatting seems a bit like a waste of time. |
Pull request was converted to draft
a858b2d to
5d0f6e4
Compare
Signed-off-by: Alexander Droste <alexander.droste@protonmail.com>
5d0f6e4 to
4c043ce
Compare
Merging this PR will improve performance by 12.16%
Performance Changes
Comparing Footnotes
|
What changes are included in this PR?
Formats all CUDA code. The clang-format style picked is identical to the style used for vortex-duckdb. Generated code is excluded.
What is the rationale for this change?
Consistent formatting for all CUDA code.
Are there any user-facing changes?
No