Skip to content

Conversation

@fivetran-kwoodbeck
Copy link
Collaborator

@github-actions
Copy link
Contributor

github-actions bot commented Dec 29, 2025

SQLGlot Integration Test Results

Comparing:

  • this branch (sqlglot:feature/annotatetype-stddev, sqlglot version: feature/annotatetype-stddev)
  • baseline (main, sqlglot version: 28.5.1.dev45)

⚠️ Limited to dialects: snowflake

By Dialect

dialect main sqlglot:feature/annotatetype-stddev difference links
snowflake -> duckdb 626/1085 passed (57.7%) 626/1085 passed (57.7%) No change full result / delta
snowflake -> snowflake 981/1085 passed (90.4%) 981/1085 passed (90.4%) No change full result / delta

Overall

main: 2170 total, 1607 passed (pass rate: 74.1%), sqlglot version: 28.5.1.dev45

sqlglot:feature/annotatetype-stddev: 2170 total, 1607 passed (pass rate: 74.1%), sqlglot version: feature/annotatetype-stddev

Difference: No change

Copy link
Collaborator

@VaggelisD VaggelisD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice & easy, just one comment to consider, feel free to merge afterwards:

Copy link
Collaborator

@VaggelisD VaggelisD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more comment:

@VaggelisD
Copy link
Collaborator

This PR is not breaking given that the type annotations exist

@VaggelisD VaggelisD changed the title feat(snowflake)!: Type annotation for Snowflake's STDDEV / STDDEV_SAMP, STDDEV_POP Chore(snowflake): Adding type annotation tests for Snowflake's STDDEV / STDDEV_SAMP, STDDEV_POP Dec 30, 2025
@fivetran-kwoodbeck fivetran-kwoodbeck merged commit 2f8ffcf into main Dec 30, 2025
9 checks passed
@fivetran-kwoodbeck fivetran-kwoodbeck deleted the feature/annotatetype-stddev branch December 30, 2025 16:25
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