Skip to content

Fixing bug with ? : not parsing, Add Sql Formatter option to scripting#170

Open
kenlassesen wants to merge 2 commits intosethreno:masterfrom
kenlassesen:master
Open

Fixing bug with ? : not parsing, Add Sql Formatter option to scripting#170
kenlassesen wants to merge 2 commits intosethreno:masterfrom
kenlassesen:master

Conversation

@kenlassesen
Copy link

The logical ordering of ? : ? failed to parse correctly in VS2019 on windows. Added ( ) to resolve issue

@kenlassesen
Copy link
Author

Added a Pretty Printer for scripting. This SQL Reformatter supports the following options (read from sqlformat.json). If this file is missing, it is created from the default values.
"Options":{
"KeywordCasing":1,
"IndentationSize":4,
"IncludeSemicolons":false,
"AlignColumnDefinitionFields":true,
"NewLineBeforeFromClause":true,
"NewLineBeforeWhereClause":true,
"NewLineBeforeGroupByClause":true,
"NewLineBeforeOrderByClause":true,
"NewLineBeforeHavingClause":true,
"NewLineBeforeJoinClause":true,
"NewLineBeforeOffsetClause":true,
"NewLineBeforeOutputClause":true,
"AlignClauseBodies":true,
"MultilineSelectElementsList":true,
"MultilineWherePredicatesList":true,
"IndentViewBody":false,
"MultilineViewColumnsList":true,
"AsKeywordOnOwnLine":true,
"IndentSetClause":false,
"AlignSetClauseItem":true,
"MultilineSetClauseItems":true,
"MultilineInsertTargetsList":true,
"MultilineInsertSourcesList":true,
"NewLineBeforeOpenParenthesisInMultilineList":false,
"NewLineBeforeCloseParenthesisInMultilineList":true
}
}

@kenlassesen kenlassesen changed the title Fixing bug with ? : not parsing, added () Fixing bug with ? : not parsing, Add Sql Formatter option to scripting Jun 14, 2019
@mryanmurphy
Copy link

I just discovered this tool today and did the same fix on my own fork. Splitting the build fix commit and reformat feature commit to separate PRs may make them easier to be pulled here.

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.

2 participants