Skip to content

docs: describe Polyglot as inspired by SQLGlot, not a port of it - #180

Merged
kvlonge merged 1 commit into
mainfrom
docs/polyglot-inspired-by
Nov 11, 2026
Merged

kvlonge merged 1 commit into
mainfrom
docs/polyglot-inspired-by

Conversation

@kvlonge

@kvlonge kvlonge commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Why

Polyglot describes itself as "a Rust SQL transpiler inspired by SQLGlot". The README, the credits page and the syntax notes called it a Rust port of SQLGlot and said SQLQuack's parser follows SQLGlot's design, which isn't accurate.

Changes

  • README: the intro sentence now uses Polyglot's own description, and the SQLGlot credit reads "for years of work on SQL transpilation that this space builds on".
  • Credits page: the same SQLGlot credit, keeping the note that SQLQuack uses the Snowflake SQL of SQLGlot's test suite as case inputs, with answers recorded from Snowflake.
  • docs/syntax-notes.md: removes three "ported from" or "based on" sqlglot remarks. The rustdoc comments they were kept from no longer carry them.

THIRD_PARTY_NOTICES already says "inspired by sqlglot" and keeps sqlglot's licence text, so it is unchanged. No code, dependency or Cargo change.

Verification

  • The website builds locally, and the built credits page shows the new text with no "port".
  • rg finds no remaining port or based-on claim about SQLGlot in the README, docs, website sources or crates.

Polyglot describes itself as a Rust SQL transpiler inspired by SQLGlot.
The README, the credits page and the syntax notes called it a Rust port
of SQLGlot and said SQLQuack's parser follows SQLGlot's design; those
claims are removed. The credits keep SQLGlot's test suite as the source
of the Snowflake case inputs.
@kvlonge
kvlonge enabled auto-merge November 11, 2026 15:26
@kvlonge
kvlonge added this pull request to the merge queue Nov 11, 2026
Merged via the queue into main with commit 1880a9e Nov 11, 2026
8 of 9 checks passed
@kvlonge
kvlonge deleted the docs/polyglot-inspired-by branch November 11, 2026 16:31
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.

1 participant