Skip to content

fix(pre-commit): normalize SPDX handling#2318

Merged
rapids-bot[bot] merged 1 commit into
NVIDIA:mainfrom
bdice:normalize-spdx-pre-commit
Jul 13, 2026
Merged

fix(pre-commit): normalize SPDX handling#2318
rapids-bot[bot] merged 1 commit into
NVIDIA:mainfrom
bdice:normalize-spdx-pre-commit

Conversation

@bdice

@bdice bdice commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Description

Contributes to rapidsai/build-planning#303.

Standardizes clang-format's SPDX protection and ensures the repository uses rapidsai/pre-commit-hooks v1.6.0. It also restores malformed SPDX headers where present.

Validated with pre-commit run --all-files.

Signed-off-by: Bradley Dice <bdice@bradleydice.com>
@bdice bdice added bug Something isn't working non-breaking Introduces a non-breaking change labels Jul 10, 2026
@copy-pr-bot

copy-pr-bot Bot commented Jul 10, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@bdice bdice marked this pull request as ready for review July 11, 2026 00:58
@bdice bdice requested a review from a team as a code owner July 11, 2026 00:58
@jameslamb

Copy link
Copy Markdown
Member

One test failure:

[==========] 9 tests from 3 test suites ran. (1069 ms total)
[  PASSED  ] 8 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] silhouetteScore.BatchedStreamPoolOrdering

 1 FAILED TEST
CMake Error at run_gpu_test.cmake:26 (execute_process):
  execute_process failed command indexes:

    1: "Child return code: 1"

Errors while running CTest

97% tests passed, 1 tests failed out of 34

Total Test time (real) = 6280.02 sec

The following tests FAILED:
	 34 - STATS_TEST (Failed)

(build link)

I've triggered a re-run, hopefully we'll see it pass.

@jameslamb

Copy link
Copy Markdown
Member

/merge

@rapids-bot rapids-bot Bot merged commit 0a093f7 into NVIDIA:main Jul 13, 2026
165 of 167 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants