Skip to content

chore: add local pre-commit check script - #4084

Open
rohitrango wants to merge 1 commit into
NVIDIA-NeMo:mainfrom
rohitrango:chore/add-precommit-check-script
Open

chore: add local pre-commit check script#4084
rohitrango wants to merge 1 commit into
NVIDIA-NeMo:mainfrom
rohitrango:chore/add-precommit-check-script

Conversation

@rohitrango

Copy link
Copy Markdown
Contributor

Summary

  • add a single local entry point for GPU-free CI checks and optional unit tests
  • support selecting, skipping, and configuring individual check stages
  • compare copyright headers against a configurable base branch

Tests

  • bash -n tools/precommit-check.sh
  • tools/precommit-check.sh --only lint --base nv/main
  • tools/precommit-check.sh --only pyrefly --base nv/main
  • tools/precommit-check.sh --only copyright --base nv/main
  • mocked unit-stage command construction with the test dependency group and extra forwarding

Signed-off-by: rohitrango <rohit.rango@gmail.com>
@rohitrango
rohitrango requested a review from a team as a code owner September 10, 2026 19:02
@copy-pr-bot

copy-pr-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rohitrango rohitrango added the CI:Lfast Runs a fast test suite and re-use nightly `main` container (but sync dependencies to PRs version) label Sep 10, 2026
@rohitrango

Copy link
Copy Markdown
Contributor Author

/ok to test a6a2f5d

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

Labels

CI:Lfast Runs a fast test suite and re-use nightly `main` container (but sync dependencies to PRs version)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant