-
Notifications
You must be signed in to change notification settings - Fork 239
chore: bump sqlformat from 0.3.5 to 0.4.0 #5441
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
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]>
@claude fix the test expectations; iterate with CI until CI passes |
Claude finished @max-sixty's task —— View job ✅ Fixed test expectations for sqlformat 0.4.0CI 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:
Summary of Changes (3 commits):
Test Status Progress:
The main formatting changes from sqlformat 0.4.0:
🔗 View CI runs | |
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
Bumps sqlformat from 0.3.5 to 0.4.0.
Changelog
Sourced from sqlformat's changelog.
Commits
f910e6f
Version 0.4.0b251c7c
fix: Place a whitespace between the array type specifier and a reserved word7d3ae8c
feat: Consistently space blocks after arguments1ce0359
feat: Correctly inline opening parentheses (#100)d20763c
Support more conditionally top level tokens (#99)227688f
chore(deps): bump actions/checkout from 4 to 5a14692e
docs: update the Readme to be more usefulfd28063
docs: add rules for Cursor7b9d239
chore(deps): update criterion requirement in the rust-dependencies group5e7146f
ci: add code coverage and modernize cachingDependabot 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)