Skip to content

Conversation

@erindru
Copy link
Collaborator

@erindru erindru commented Dec 8, 2025

This PR:

  • Introduces automatic triggers for the integration tests. There are two ways:
    • Use a /integration-tests indicator in your PR description or comment. For a specific dialect or dialects, use /integration-tests dialects=duckdb,bigquery
    • If there is no /integration-tests indicator, then the script looks at the changed files. If any of the file names contain a dialect supported by the test suite, the tests will run for that dialect
    • It's based on a Python script that inspects the GitHub event payload and writes specific output variables depending on what it finds
  • Updates the summary comment if it already exists rather than publishing a new one each time

/integration-tests dialect=bigquery

@erindru erindru force-pushed the erin/integration-test-automation branch 4 times, most recently from 790ecbb to fd59147 Compare December 8, 2025 23:29
@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

SQLGlot Integration Test Results

Comparing:

  • this branch (sqlglot:erin/integration-test-automation, sqlglot version: erin/integration-test-automation)
  • baseline (main, sqlglot version: 28.1.1.dev71)

⚠️ Limited to dialects: bigquery

By Dialect

dialect main sqlglot:erin/integration-test-automation difference links
bigquery -> bigquery 651/651 passed (100.0%) 651/651 passed (100.0%) No change full result / delta
bigquery -> duckdb 459/650 passed (70.6%) 458/650 passed (70.5%) ⬇ regressed by 0.2% full result / delta

Overall

main: 1301 total, 1110 passed (pass rate: 85.3%), sqlglot version: 28.1.1.dev71

sqlglot:erin/integration-test-automation: 1301 total, 1109 passed (pass rate: 85.2%), sqlglot version: erin/integration-test-automation

Difference: ⬇ regressed by 0.1%

@erindru erindru force-pushed the erin/integration-test-automation branch 3 times, most recently from 0141945 to 0c098e9 Compare December 10, 2025 03:52
@erindru erindru marked this pull request as ready for review December 10, 2025 03:54
@erindru erindru force-pushed the erin/integration-test-automation branch from 4213ab5 to 8311c8b Compare December 10, 2025 19:58
@VaggelisD VaggelisD merged commit ee5e7b9 into main Dec 11, 2025
9 checks passed
@VaggelisD VaggelisD deleted the erin/integration-test-automation branch December 11, 2025 09: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.

3 participants