Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Python] add more sql indicators #3814

Merged
merged 2 commits into from
Aug 2, 2023
Merged

[Python] add more sql indicators #3814

merged 2 commits into from
Aug 2, 2023

Conversation

keith-hall
Copy link
Collaborator

MERGE INTO is ANSI SQL: https://en.wikipedia.org/wiki/Merge_(SQL)#Usage BEGIN works for MySQL and TSQL

@keith-hall
[Python] add more sql indicators
7963492 
`MERGE INTO` is ANSI SQL: https://en.wikipedia.org/wiki/Merge_(SQL)#Usage
`BEGIN` works for MySQL and TSQL
@deathaxe
Copy link
Collaborator

deathaxe commented Aug 2, 2023

Could you add the change to PHP Source as well?

@keith-hall
[PHP] add more sql indicators
a7c2109 
to keep it in sync with Python
@keith-hall
Copy link
Collaborator Author

Could you add the change to PHP Source as well?

Done

@jfcherng jfcherng merged commit 80e4147 into sublimehq:master Aug 2, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deathaxe deathaxe deathaxe approved these changes

@jfcherng jfcherng jfcherng approved these changes

@michaelblyons michaelblyons michaelblyons approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@keith-hall @deathaxe @michaelblyons @jfcherng