people/f0rk/format sql #1
Conversation
|
On f0rk@0114527 - this makes no sense. FROM and subsequent JOINs describe a single source. They have to all be indented at the same level. |
|
"Have" is a strong word as I don't believe a parser will care about the indentation, it's only there for readability. At any rate, frew asked me to add command-line switches to control this behavior so that will go back to the way it originally was. You can close this pull request if you'd like and I'll open a new one when I'm done with the requested changes. |
|
Well, just to be clear, I meant make it configurable, note that the formatter isn't just used from the little script included with the dist, but that would become obvious when you started to implement it anyway |
|
I am closing this PR given the above answers, please feel free to continue the discussion here or on the mailing list. |
|
Yeah, a sep dist would be fine with me. |
This branch modifies the way queries are formatted by unparse so that when using format-sql the output is (IMHO) more readable. This also modifies the behavior of format-sql to slurp all of its input, instead of on a line by line basis.