ci: replace SPDX header with LICENSE header on L0_torch_aoti_hstu (TRI-1654) - #8942
Merged
Merged
Conversation
…TRI-1654) check_copyright.py rejects the two-line SPDX header on qa/L0_torch_aoti_hstu/test.sh, failing the nightly L0_copyrights job. Convert to the long-form BSD LICENSE header, matching 82cf959. Signed-off-by: Akhil <asaraswathi@nvidia.com>
Greptile SummaryThis PR replaces the SPDX metadata in
Confidence Score: 5/5The PR appears safe to merge because it changes only comment-form licensing text and preserves the script’s behavior. The shebang remains first, and every changed line is a shell comment containing the replacement license header; no functional failure remains. Important Files Changed
Reviews (1): Last reviewed commit: "ci: replace SPDX header with LICENSE hea..." | Re-trigger Greptile |
mudit-eng
approved these changes
Aug 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does the PR do?
Converts the two-line SPDX header on
qa/L0_torch_aoti_hstu/test.shto the long-form BSD LICENSE header thatqa/common/check_copyright.pyrequires. This was the last remaining copyright violation failing the nightlyL0_copyrightsjob.Checklist
<commit_type>: <Title>Commit Type:
Related PRs:
Part of TRI-1654. Companion changes already merged:
#8928— copyright header onqa/L2_build_presets/requirements.txt!1855— copyright headers on the RHEL tutorial testsWhere should the reviewer start?
qa/L0_torch_aoti_hstu/test.sh— the header swap is the entire diff (+25/-2).Test plan:
python3 qa/common/check_copyright.py --year=2026 *→ exit 0 (was exit 1 on this file). Last remaining violation, so the nightlyL0_copyrightsgoes green.Caveats:
None — header-only change, no functional or behavioral impact.
Background
The file was added (#8907) with a two-line SPDX header.
check_copyright.pyonly accepts the long-form BSD header, and theadd-licensepre-commit hook doesn't convert SPDX (it sees the SPDX copyright line as already present), so it slipped through and broke the nightly. Same class of fix as82cf9596.Related Issues: