Skip to content

Remove Faker fill and preserve strict PARSE semantics - #43

Merged
skadel merged 2 commits into
mainfrom
codex/remove-faker-preserve-parse-semantics
Jul 26, 2026
Merged

Remove Faker fill and preserve strict PARSE semantics#43
skadel merged 2 commits into
mainfrom
codex/remove-faker-preserve-parse-semantics

Conversation

@skadel

@skadel skadel commented Jul 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove Faker-based filling so all generated values stay coherent with the LLM-authored scenario
  • remove the unused Faker dependency and obsolete regression coverage
  • preserve strict BigQuery PARSE_* behavior while keeping SAFE.PARSE_* tolerant
  • update sqlglot 30.12 canaries and scalar-folding expectations

Validation

  • poetry run pytest -q --disable-warnings — 2095 passed, 1 xfailed
  • focused parsing/generator tests — 80 passed
  • Ruff checks passed
  • git diff --check passed

@skadel
skadel merged commit f3db440 into main Jul 26, 2026
2 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