doctrine/sql-formatter

Feature request: uppercase SQL keywords

greg0ire opened this issue · 4 comments

ftrotter commented

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!!

  • 👍1
greg0ire commented

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?

rodnaph commented

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

greg0ire commented

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

  • 👍1