search

barklan / inline_sql_syntax

Highlight and lint inline SQL strings.
https://marketplace.visualstudio.com/items?itemName=qufiwefefwoyn.inline-sql-syntax
MIT License
57 stars 24 forks source link

Enhanced auto-syntax highlighting #42

Open sepsol opened 2 years ago

sepsol commented 2 years ago

Currently, the extension automatically detects strings starting with certain words as sql syntax but this wouldn't work if the string first starts with tabs, spaces, and line breaks and then has those words. It'd be great if it could skip whitespace characters and check the first valid word for syntax highlighting.

ShimShamSam commented 2 years ago

It would be even nicer if we could set our own regexp/startswith config. For instance, the keyword "WITH" should trigger highlighting too.

RLaursen commented 2 years ago

TextMate does not support multiple line matches, so this could be a significant change, but desirable as I start queries with `\n

macnewbold commented 1 year ago

I ran into the issue with WITH today as well, and would love it if that were on the list of SQL-start keywords. This also matches up nicely with the recently-submitted issue https://github.com/barklan/inline_sql_syntax/issues/109, regarding whitespace at the beginning of the string before a magic keyword. We regularly put a large inline query at the start of the next line, so having whitespace be ignored (and recognizing WITH) would save us a whole lot of --sql sprinkled everywhere.