sql-formatter icon indicating copy to clipboard operation
sql-formatter copied to clipboard

Feature request: uppercase SQL keywords

Open greg0ire opened this issue 4 years ago • 4 comments

See https://github.com/jdorn/sql-formatter/issues/97 and https://github.com/jdorn/sql-formatter/issues/84

greg0ire avatar Apr 11 '20 09:04 greg0ire

Thanks for transferring this over!! I am still interested. Generally, I think this should be a configuration signal and there should be the following options:

  • Do not change the case in the original SQL at all.
  • Change all SQL keywords to upper-case
  • Change all SQL keywords and SQL functions to upper-case

Thanks for looking into it!!

ftrotter avatar Jul 31 '20 04:07 ftrotter

should be a configuration signal

I think we should change the constructor of SqlFormatter, but I'm not sure how… maybe pass an options array? @goetas what would you recommend for this?

greg0ire avatar Jul 31 '20 17:07 greg0ire

@greg0ire Linked PR has a patch for this, any chance of a review? I'd like to use this, thanks.

rodnaph avatar Aug 19 '21 13:08 rodnaph

@rodnaph didn't spot it in my notifications, but sure!

greg0ire avatar Aug 19 '21 14:08 greg0ire