Skip to content

ci: run SQLite regressions on pushes and pull requests - #4

Merged
th3nolo merged 2 commits into
mainfrom
ci/sqlite-regressions
Sep 18, 2026
Merged

th3nolo merged 2 commits into
mainfrom
ci/sqlite-regressions

Conversation

@th3nolo

@th3nolo th3nolo commented Sep 18, 2026 •

Copy link
Copy Markdown
Owner

The 12 SQLite regression tests were not running automatically, so changes to mutation protection, SQL literal handling, or result comparison could land without executing them. Add push and pull-request CI that runs the existing suite against temporary fixture databases, plus Python syntax and commit-whitespace checks.

Pin Python 3.13.7, uv 0.11.19 (release SHA-256 verified), and both actions by full commit SHA. The suite uses only the standard library; no benchmark dependencies, credentials, provider calls, or dataset downloads are needed. README documents the exact commands and fixture-only scope.

Validation:

  • All 12 existing tests passed locally on Python 3.13.7 using uv 0.11.19.
  • Python compileall and git diff --check passed. No formatter, linter, or type checker is configured in this repository.
  • In a disposable copy, replacing the comparator with an always-true implementation caused the same unittest command to report 22 assertion failures and exit 1; repository implementation and tests were unchanged.
  • Tool release identities and dates were verified against upstream GitHub metadata; all selected releases are older than 72 hours. No Python dependencies or lockfiles changed.

These checks exercise the real SQLite evaluator/comparator with small local fixtures. They do not establish measured model accuracy or benchmark-native acceptance.

GitHub validation at final head 34eee79e3ca797da0aeafc4e60212abc7f887ce8:

  • Push CI: passed.
  • Pull-request CI: passed; logs confirm all 12 tests executed and passed, plus syntax and whitespace checks.

@th3nolo
th3nolo merged commit 7eccdc4 into main Sep 18, 2026
3 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.

1 participant