Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add more granular operator priorities to capture order of operations #200

Closed
tconbeer opened this issue Jul 14, 2022 · 1 comment · Fixed by #244
Closed

Add more granular operator priorities to capture order of operations #200

tconbeer opened this issue Jul 14, 2022 · 1 comment · Fixed by #244
Labels
enhancement New feature or request
Milestone
v0.11.0

Comments

@tconbeer
Copy link
Owner

e.g., today this won't merge onto a single line, since it's too long, so we leave it:

foofoofoofoofoofoofoofoofoofoo
/ barbarbarbarbarbarbar
+ foofoofoofoofoofoofoofoofoofoo
/ bazbazbazbazbazbaz
+ foofoofoofoofoofoofoofoofoofoo
/ quxquxquxquxqux

but it would be better to merge the / lines:

foofoofoofoofoofoofoofoofoofoo / barbarbarbarbarbarbar
+ foofoofoofoofoofoofoofoofoofoo / bazbazbazbazbazbaz
+ foofoofoofoofoofoofoofoofoofoo / quxquxquxquxqux
@tconbeer tconbeer added the enhancement New feature or request label Jul 14, 2022
@tconbeer tconbeer added this to the v0.11.0 milestone Aug 2, 2022
@tconbeer
Copy link
Owner Author

https://www.postgresql.org/docs/current/sql-syntax-lexical.html#SQL-PRECEDENCE

tconbeer added a commit that referenced this issue Aug 17, 2022
@tconbeer
feat: add more granular operator precedence, close #200
0f0fa51
@tconbeer tconbeer mentioned this issue Aug 17, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
v0.11.0
Development

Successfully merging a pull request may close this issue.

feat/issue 200/order of operations tconbeer/sqlfmt
1 participant
@tconbeer