Skip to content

Add automatic header checks and formatting#14

Merged
arjkesh merged 1 commit intoNVIDIA:mainfrom
arjkesh:header_check
Dec 19, 2025
Merged

Add automatic header checks and formatting#14
arjkesh merged 1 commit intoNVIDIA:mainfrom
arjkesh:header_check

Conversation

@arjkesh
Copy link
Copy Markdown
Collaborator

@arjkesh arjkesh commented Dec 18, 2025

Description

  • Add new utility script to check for license headers, as well as automatically write them out if they don't exist
  • Update format.sh to run this action in addition to ruff
  • Update infra tests to run the --check action to confirm that headers exist

CI Configuration

config:
  build: false
  # valid options are "ops" and "benchmark"
  test: []

Checklist

  • Code formatted and imports sorted via repo specifications (./format.sh)
  • Documentation updated (if needed)
  • CI configuration reviewed

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Dec 18, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@arjkesh
Copy link
Copy Markdown
Collaborator Author

arjkesh commented Dec 18, 2025

/ok to test bc6a980

Copy link
Copy Markdown
Collaborator

@hannahli-nv hannahli-nv left a comment

Choose a reason for hiding this comment

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

LGTM, thx!

@arjkesh arjkesh merged commit e750ff4 into NVIDIA:main Dec 19, 2025
9 checks passed
@arjkesh arjkesh deleted the header_check branch December 19, 2025 04:56
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.

2 participants