Skip to content

ci: Dependabot, eval-report artifacts, and assertion-count doc fix#14

Merged
amar-python merged 2 commits into
mainfrom
claude/csv-loading-postgres-migration-qvkl4l
Jul 19, 2026
Merged

ci: Dependabot, eval-report artifacts, and assertion-count doc fix#14
amar-python merged 2 commits into
mainfrom
claude/csv-loading-postgres-migration-qvkl4l

Conversation

@amar-python

Copy link
Copy Markdown
Owner

Summary

Two follow-ups to the quality gate (PR #2):

ci: add Dependabot config and preserve eval reports as CI artifacts

  • .github/dependabot.yml: weekly update PRs for pip (requirements-dev.txt) and GitHub Actions versions.
  • Both quality-gate jobs now upload evals/reports/ (including the per-run VCRM gap report) as artifacts with if: always(), 30-day retention — previously the reports were generated in CI and discarded on teardown.

docs: correct SQL assertion count from 85 to 142

  • README (badge + 4 text spots), ARCHITECTURE.md, and a workflow comment claimed 85 assertions; the suite runs 142. The stale claim dates to the initial release and was unnoticeable while the suites never executed.
  • Tightens the Tier S eval expectation (min_total_assertions 85 → 142) so a future drop in assertion coverage fails the eval instead of passing silently.

Test plan

  • Full eval tiers P/I/S verified 25/25 against live PostgreSQL 16 with the tightened expectation
  • Both YAML files validate
  • Three required checks green on this PR

claude added 2 commits July 19, 2026 13:43
- .github/dependabot.yml: weekly update PRs for pip (requirements-dev.txt)
  and the GitHub Actions used by the workflows.
- quality-gate.yml: both jobs now upload evals/reports/ (including the
  per-run VCRM gap report) as artifacts with if: always(), so the audit
  trail survives the runner instead of being discarded — previously the
  reports were generated in CI and lost on job teardown. 30-day retention.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VBrxqChRJtxdvSpFhiUWUy
The suite has grown to 142 assertions but README (badge, features list,
coverage section, checklist), ARCHITECTURE.md, and a workflow comment
still said 85 — a claim dating to the initial release that went
unnoticed while the suites never executed. Verified against a live
deploy: 142/142.

Also tightens the Tier S eval expectation (min_total_assertions 85 ->
142) so a future drop in assertion count fails the eval instead of
passing silently.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VBrxqChRJtxdvSpFhiUWUy
@amar-python
amar-python merged commit 785f83e into main Jul 19, 2026
4 checks passed
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