Skip to content

feat: add support for SQL syntax highlighting - #943

Open
Lakshay Chauhan (nos1dot618) wants to merge 1 commit into
microsoft:mainfrom
nos1dot618:feat/942-sql-syntax-highlighting
Open

feat: add support for SQL syntax highlighting#943
Lakshay Chauhan (nos1dot618) wants to merge 1 commit into
microsoft:mainfrom
nos1dot618:feat/942-sql-syntax-highlighting

Conversation

@nos1dot618

Copy link
Copy Markdown

This adds initial LSH syntax highlighting support for SQL.
It also updates Markdown fenced code block routing accordingly.

Scope

This is a lightweight LSH-only change with no new dependencies.

The changes are limited to syntax definitions and highlighting fixtures:

  • New LSH definition for SQL.
  • Markdown fenced code block routing update.
  • New SQL highlighting test sample.
image

Validation

  • cargo test -q

Notes

@nos1dot618

Copy link
Copy Markdown
Author

Hey Leonard Hecker (@lhecker), Dustin L. Howett (@DHowett), could you please review this PR.
Thanks

@nos1dot618
Lakshay Chauhan (nos1dot618) force-pushed the feat/942-sql-syntax-highlighting branch 2 times, most recently from 8ea6729 to 9d00a58 Compare August 28, 2026 21:29
@nos1dot618 Lakshay Chauhan (nos1dot618) changed the title feat: Add support for SQL syntax highlighting feat: add support for SQL syntax highlighting Aug 28, 2026
@nos1dot618
Lakshay Chauhan (nos1dot618) force-pushed the feat/942-sql-syntax-highlighting branch 2 times, most recently from 530a363 to 485e2e6 Compare August 28, 2026 21:49
@nos1dot618
Lakshay Chauhan (nos1dot618) force-pushed the feat/942-sql-syntax-highlighting branch from 485e2e6 to c5cd3fc Compare August 28, 2026 21:50
@lhecker

Copy link
Copy Markdown
Member

…you opened the PR hours ago, why do you tag maintainers on it?

@nos1dot618

Lakshay Chauhan (nos1dot618) commented Aug 29, 2026

Copy link
Copy Markdown
Author

…you opened the PR hours ago, why do you tag maintainers on it?

Yeah, I understand it was unnecessary since the PR had only been open for a few hours. I'll avoid doing this from the next time.

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.

feat: add support for SQL syntax highlighting

2 participants