Skip to content

Add repository validation and restore VSS workshop ingress manifest#8

Merged
azeltov merged 2 commits into
mainfrom
chore/add-pre-commit-checks
Jul 9, 2026
Merged

Add repository validation and restore VSS workshop ingress manifest#8
azeltov merged 2 commits into
mainfrom
chore/add-pre-commit-checks

Conversation

@harmke

@harmke harmke commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add repository-wide quality checks and restore the ingress manifest referenced by the VSS workshop.

Changes include:

  • Add pre-commit checks for whitespace, line endings, merge conflicts, YAML, Markdown formatting, and documentation links.
  • Add a GitHub Actions workflow that runs pre-commit on pull requests.
  • Pin pre-commit to the tested version.
  • Add timeout and retry handling for external link validation.
  • Add generic repository guidance in AGENTS.md.
  • Restore the missing VSS ingress manifest from repository history.
  • Fix existing Markdown formatting and broken or case-mismatched links.
  • Remove the PR template link to the nonexistent CLA.MD; DCO sign-off remains required.
  • Document local validation commands for contributors.

Related Issue(s)

N/A

Type of Change

  • New sample
  • Update to existing sample
  • Bug fix
  • Documentation
  • Infrastructure / CI
  • Other (describe below)

Testing

The following checks pass:

pre-commit run --all-files
git diff --check

This includes:

  • YAML validation
  • Markdown linting
  • Internal and external link validation
  • Whitespace and line-ending checks
  • Merge-conflict marker detection

The full pre-commit suite was run repeatedly and produced no additional changes.

Checklist

  • My commits are signed off (git commit -s)
  • I have read CONTRIBUTING.md
  • I have updated documentation where relevant
  • I have verified the affected sample runs end-to-end, or explained why that was not possible

harmke added 2 commits July 8, 2026 11:06
Signed-off-by: Harmke Alkemade <halkemade@nvidia.com>
Signed-off-by: Harmke Alkemade <halkemade@nvidia.com>
@harmke harmke requested review from antonslutskyms and azeltov July 8, 2026 09:15

@azeltov azeltov left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

i like the new pre-commit changes. Lets use it.

@azeltov azeltov merged commit 0ab81a0 into main Jul 9, 2026
1 check passed
@azeltov azeltov deleted the chore/add-pre-commit-checks branch July 9, 2026 17:26
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