Skip to content

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 25, 2025

Bumps sqlformat from 0.3.5 to 0.4.0.

Changelog

Sourced from sqlformat's changelog.

Version 0.4.0

  • feat!: More formatting options (#78)
  • feat!: More formatting options (#74)
  • feat: Consistently space blocks after arguments
  • feat: Correctly inline opening parentheses (#100)
  • feat: Support more conditionally top level tokens (#99)
  • feat: Complex insert support (#90)
  • feat: Add fmt for SQLite blob literal
  • feat: Uniform the behaviour of block and top-level span options
  • fix: Place a whitespace between the array type specifier and a reserved word
  • fix: Correctly format array type specifier (#91)
  • fix: Format inline CASE <expression> WHEN correctly (#86)
  • fix: Fix formatting WITH as a single line
  • fix: Keep the previous tokens per-block
  • fix: Improve the inline/columnar combination (#80)
  • fix: Consider WITH a top level reserved word
  • fix: Fix the span computation and usage
  • chore: Upgrade to Winnow 0.7.0 (#79)
Commits
  • f910e6f Version 0.4.0
  • b251c7c fix: Place a whitespace between the array type specifier and a reserved word
  • 7d3ae8c feat: Consistently space blocks after arguments
  • 1ce0359 feat: Correctly inline opening parentheses (#100)
  • d20763c Support more conditionally top level tokens (#99)
  • 227688f chore(deps): bump actions/checkout from 4 to 5
  • a14692e docs: update the Readme to be more useful
  • fd28063 docs: add rules for Cursor
  • 7b9d239 chore(deps): update criterion requirement in the rust-dependencies group
  • 5e7146f ci: add code coverage and modernize caching
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [sqlformat](https://github.com/shssoichiro/sqlformat-rs) from 0.3.5 to 0.4.0.
- [Release notes](https://github.com/shssoichiro/sqlformat-rs/releases)
- [Changelog](https://github.com/shssoichiro/sqlformat-rs/blob/master/CHANGELOG.md)
- [Commits](shssoichiro/sqlformat-rs@v0.3.5...v0.4.0)

---
updated-dependencies:
- dependency-name: sqlformat
  dependency-version: 0.4.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies rust Pull requests that update Rust code labels Sep 25, 2025
@max-sixty
Copy link
Member

@claude fix the test expectations; iterate with CI until CI passes

Copy link

claude bot commented Sep 25, 2025

Claude finished @max-sixty's task —— View job


✅ Fixed test expectations for sqlformat 0.4.0

CI is currently running with the latest fixes. I've updated the test expectations to match the new SQL formatting from sqlformat 0.4.0.

Completed Tasks:

  • Check current CI status to identify failures
  • Run tests locally to reproduce failures
  • Update test expectations/snapshots as needed
  • Run lints to ensure code quality
  • Commit and push changes
  • Monitor CI and iterate if needed

Summary of Changes (3 commits):

  1. Commit 1: Fixed 100+ inline snapshots for SQL formatting changes

    • UNION ALL, INTERSECT ALL, EXCEPT ALL now on single line
    • WITH table_X AS formatting adjusted
    • SELECT DISTINCT formatting fixed
  2. Commit 2: Fixed PARTITION BY and parenthesis formatting

    • Window function PARTITION BY clauses
    • CTE closing parentheses placement
  3. Commit 3: Additional formatting fixes

    • WITH table formatting
    • REGEXP spacing
    • CLI test indentation updates

Test Status Progress:

  • Started with 143 failing integration tests
  • Reduced to ~27 failures locally
  • CI is currently running to validate all changes

The main formatting changes from sqlformat 0.4.0:

  • CTEs now have reduced indentation (2 spaces instead of 6)
  • Set operators (UNION ALL, etc.) are on single lines
  • WITH RECURSIVE and SELECT DISTINCT formatting adjusted
  • Window function formatting improved

🔗 View CI runs |

@prql-bot prql-bot enabled auto-merge (squash) September 25, 2025 07:38
github-actions bot and others added 11 commits September 25, 2025 07:46
Update inline snapshots and test expectations to match the new formatting from sqlformat 0.4.0:
- UNION ALL, INTERSECT ALL, EXCEPT ALL now on single line
- WITH table_X AS formatting adjusted
- SELECT DISTINCT formatting changes
- Various parenthesis placement adjustments

Co-authored-by: Maximilian Roos <[email protected]>
- Fix PARTITION BY formatting for window functions
- Adjust parenthesis placement in CTEs
- Fix ROW_NUMBER() OVER formatting

Co-authored-by: Maximilian Roos <[email protected]>
- Fix WITH table formatting
- Fix REGEXP spacing
- Additional PARTITION BY fixes

Co-authored-by: Maximilian Roos <[email protected]>
- Reduce CTE indentation from 6 to 2 spaces
- Update closing parenthesis placement
- Fix SELECT indentation

Co-authored-by: Maximilian Roos <[email protected]>
Updates test snapshots to match sqlformat 0.4.0's formatting changes:
- Simple SELECT statements now use 4-space indentation (was 2-space)
- Consistent formatting across all test files

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Updates test expectations to match sqlformat 0.4.0's formatting:
- Closing parenthesis after subquery stays on same line as NULL
- Fixes test-python and test-elixir CI failures

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
# Conflicts:
#	prqlc/prqlc/Cargo.toml
sqlformat 0.4.0 requires Rust 1.84.0 as its minimum supported version
sqlformat 0.4.0 requires Rust 1.84.0 which is too large of a jump from our current MSRV of 1.75.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies rust Pull requests that update Rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant