Skip to content

chore: apply NVIDIA SPDX license headers via pre-commit#584

Open
misiugodfrey wants to merge 5 commits into
mainfrom
misiug/UpdateAllCopyrights
Open

chore: apply NVIDIA SPDX license headers via pre-commit#584
misiugodfrey wants to merge 5 commits into
mainfrom
misiug/UpdateAllCopyrights

Conversation

@misiugodfrey

@misiugodfrey misiugodfrey commented Jun 10, 2026

Copy link
Copy Markdown

Summary

  • Adds .pre-commit-config.yaml + scripts/strip_legacy_license_header.py to enforce NVIDIA SPDX license headers going forward
  • Bulk-applies NVIDIA SPDX headers to all 161 existing Python/shell source files (none previously had headers)
  • Part of the org-wide SPDX header rollout (see heavydb-internal#8287)

Test plan

  • pre-commit run verify-copyright --all-files reports no changes after this PR lands — verified 2026-06-11
  • Follow-on PRs that touch source files get headers enforced automatically

check_license_headers.sh and strip_legacy_license_header.py are only
used by heavydb-internal. Removing them from this repo along with the
pre-commit local hook that referenced strip_legacy_license_header.py.
verify-copyright (rapidsai/pre-commit-hooks) handles ongoing enforcement.

@simoneves simoneves left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants