-
Notifications
You must be signed in to change notification settings - Fork 747
Add lexer for GoogleSQL #2820
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add lexer for GoogleSQL #2820
Conversation
Google has a dialect of SQL known as GoogleSQL (formerly ZetaSQL), used in BigQuery and other products. It is documented at https://cloud.google.com/bigquery/googlesql. This adds a GoogleSQL lexer. Resolves pygments#2814.
|
I see tests are missing, could you add some? These are the instructions: https://pygments.org/docs/lexerdevelopment/#how-to-test-your-lexer |
Done! The test is minimal, but the primary difference in GoogleSQL is a bunch of new keywords (functions), and the golden looks good for those. Thanks. |
|
Nice, thanks for being so quick! LGTM 👍 |
Thank you too. I don't know what your standards/process is for merging, but if you can merge, I have a couple followup PRs that clean up and improve sql.py, but they will definitely cause conflicts with this one so I'm waiting until after merge. |
|
Merged for 2.19, thanks! |
Google has a dialect of SQL known as GoogleSQL (formerly ZetaSQL), used in BigQuery and other products. It is documented at https://cloud.google.com/bigquery/googlesql.
Resolves #2814.