Skip to content

Guardrails: YAML lint, build-pre-test, pre-commit .perf detection#5

Closed
alderpath wants to merge 1 commit into
masterfrom
pr-guardrails
Closed

Guardrails: YAML lint, build-pre-test, pre-commit .perf detection#5
alderpath wants to merge 1 commit into
masterfrom
pr-guardrails

Conversation

@alderpath

Copy link
Copy Markdown
Contributor

Three guardrails to prevent CI failure types found in #4:

  1. YAML workflow lint — validates all .github/workflows/*.yml with python3 yaml.safe_load before test run. Catches colon-in-shell issues.
  2. Build check before test — cargo check --release runs before cargo test. Catches compile errors 5 min faster.
  3. Pre-commit stale .perf detection — blocks commits that include .perf files.
  4. Pre-commit YAML lint — same check client-side before push.

@alderpath

Copy link
Copy Markdown
Contributor Author

Closing — superseded by later commits that added YAML linting via CI step and pre-commit hook separately. This branch would remove the existing stale-file detector and gitleaks check.

@alderpath alderpath closed this Jun 16, 2026
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.

1 participant