search

joereynolds / sql-lint

An SQL linter
MIT License
437 stars 40 forks source link

Does not lint materialized views #245

Closed LydiaF closed 1 year ago

LydiaF commented 1 year ago

Hello,

Thank you for your package.

It would be great if it could lint CREATE MATERIALIZED VIEW

photo_2023-04-28_15-11-34

Thanks for considering!

joereynolds commented 1 year ago

Hey @LydiaF What dialect is that, MySQL?

LydiaF commented 1 year ago

Hi Joe,

Thanks for answering :)

I have actually since updated the settings to use postgres. However the issue is still there: Screenshot 2023-05-11 at 18 59 15

Best,

Lydia

phrocker commented 1 year ago

I added this: https://github.com/joereynolds/sql-lint/pull/246

I don't know what I'm doing in the project generally but happy to receive feedback on what I'm missing.

YNedderhoff commented 1 year ago

Just here to say that we'd love this to be included as well!

joereynolds commented 1 year ago

Hey guys! Sorry about the delay, I've just merged in #246. If you build off master you'll be able to use it. Either that or wait until I release a version (not sure when this will be if I'm honest)

Thank you!