Skip to content

[no-ci] toolshed: require all LICENSE files to match the repository license - #2762

Merged
rwgk merged 1 commit into
NVIDIA:mainfrom
rwgk:toolshed/check-all-license-files
Sep 3, 2026
Merged

[no-ci] toolshed: require all LICENSE files to match the repository license#2762
rwgk merged 1 commit into
NVIDIA:mainfrom
rwgk:toolshed/check-all-license-files

Conversation

@rwgk

@rwgk rwgk commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Description

No linked issue; this was found while working on #2737.

Make the check-spdx hook compare every LICENSE file it receives with the repository-level LICENSE. This removes the hard-coded package-license list, automatically covers future package roots, and reports a mismatch only in the pre-commit batch containing that file instead of repeating it for every batch.

Normalize cuda_pathfinder/LICENSE so all currently tracked LICENSE files are byte-identical. CI runs pre-commit with --all-files, enforcing the repository-wide invariant downstream.

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

…po are identical

This avoids:
* hard-coded LICENSE filepaths.
* checking the same LICENSE files multiple times, depending on how pre-commit batches the files.
@rwgk rwgk added this to the cuda.bindings 13.5.0 & 12.9.10 milestone Sep 3, 2026
@rwgk rwgk added the CI/CD CI/CD infrastructure label Sep 3, 2026
@rwgk rwgk self-assigned this Sep 3, 2026
@github-actions github-actions Bot added the cuda.pathfinder Everything related to the cuda.pathfinder module label Sep 3, 2026
@rwgk rwgk changed the title toolshed: require all LICENSE files to match the repository license [no-ci] toolshed: require all LICENSE files to match the repository license Sep 3, 2026
@rwgk
rwgk requested a review from mdboom September 3, 2026 17:05
@rwgk rwgk closed this Sep 3, 2026
@rwgk rwgk reopened this Sep 3, 2026

@mdboom mdboom 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.

LGTM

@rwgk
rwgk merged commit fb57dbf into NVIDIA:main Sep 3, 2026
46 of 73 checks passed
@rwgk
rwgk deleted the toolshed/check-all-license-files branch September 3, 2026 18:13
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

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

Labels

CI/CD CI/CD infrastructure cuda.pathfinder Everything related to the cuda.pathfinder module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants