Skip to content

chore: PR template + CI driven by saar dogfood#2

Merged
DevanshuNEU merged 3 commits into
mainfrom
chore/github-templates-workflows
Mar 17, 2026
Merged

chore: PR template + CI driven by saar dogfood#2
DevanshuNEU merged 3 commits into
mainfrom
chore/github-templates-workflows

Conversation

@DevanshuNEU
Copy link
Copy Markdown
Contributor

PR template and CI improvements written using what saar extracted.

  • PR template: real verification commands from AGENTS.md, dogfood step required
  • CI: lint runs once not 12x, tests depend on lint passing
  • Dogfood in CI: saar must score 100/100 on itself
  • auto-release fixed: was firing before CI, now waits for workflow_run success

…-- expand compact one-liners to multi-line"

This reverts commit 7211572.
PR template:
- Verification commands from AGENTS.md (pytest, ruff, saar extract)
- Module checklist matching saar/commands/ saar/extractors/ structure
- saar/models.py off-limits warning surfaced explicitly
- Dogfood step: saar extract . --no-interview as required check

CI improvements:
- Lint extracted to separate job (runs once, not 12x across matrix)
- Tests depend on lint passing (fail fast on obvious errors)
- Dogfood step in install-test: saar must score 100/100 on itself
- auto-release now waits for CI workflow_run completion
  (was: fires on every push, could release broken code)
  (now: only fires when CI concludes with success)
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
saar Ready Ready Preview, Comment Mar 17, 2026 4:17am

@DevanshuNEU DevanshuNEU merged commit e3f220a into main Mar 17, 2026
16 checks passed
@DevanshuNEU DevanshuNEU deleted the chore/github-templates-workflows branch March 17, 2026 04:33
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