Skip to content

ci: add copyright header to build preset requirements (TRI-1654) - #8928

Open
akhilraj9 wants to merge 1 commit into
mainfrom
asaraswathi/tri-1654-fix-l0_copyrights-job
Open

ci: add copyright header to build preset requirements (TRI-1654)#8928
akhilraj9 wants to merge 1 commit into
mainfrom
asaraswathi/tri-1654-fix-l0_copyrights-job

Conversation

@akhilraj9

@akhilraj9 akhilraj9 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

qa/L2_build_presets/requirements.txt was merged without the required NVIDIA
copyright header, which fails the nightly L0_copyrights job. This adds it.

Root cause

qa/common/check_copyright.py runs nightly across the whole repo and requires
every file to start with the NVIDIA copyright header. This file started with
distro, so the checker failed:

incorrect prefix for copyright line, allowed prefixes '# ' or '// ',
for qa/L2_build_presets/requirements.txt: distro

Fix

Prepend the standard #-prefixed BSD copyright header (2026) to the file.

Testing

python3 qa/common/check_copyright.py --year=2026 *

→ exit 0, no violations.

@greptile-apps

greptile-apps Bot commented Aug 13, 2026

Copy link
Copy Markdown

Greptile Summary

Adds the required NVIDIA copyright and BSD license header to the build-preset requirements file so it complies with the repository copyright check.

  • Prepends a comment-only 2026 copyright and license block.
  • Leaves all dependency declarations unchanged.

Confidence Score: 5/5

The PR appears safe to merge because it only adds a valid comment header and leaves dependency behavior unchanged.

The added lines are comments accepted by requirements-file syntax, while the existing distro, pytest, and requests declarations remain unchanged.

Important Files Changed

Filename Overview
qa/L2_build_presets/requirements.txt Adds the standard comment-prefixed copyright header without changing package requirements or installation behavior.

Reviews (1): Last reviewed commit: "ci: add copyright header to build preset..." | Re-trigger Greptile

@akhilraj9 akhilraj9 self-assigned this Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants