Skip to content

Conversation

@gontadu
Copy link
Contributor

@gontadu gontadu commented Jun 23, 2016

USE is used to select a database in SQL Server when a database is connected to.
EXEC and OPENQUERY are both used to get data on other servers.

I have grouped this with the VALUES keyword as EXEC/OPENQUERY are dealing with similar data.
I have also added USE into this dictionart as I felt it is cleaner than adding into dictionary keyword.other.DML.sql but let me know if you disagree.

USE is used to select a database in SQL Server when a database is connected to.
EXEC and OPENQUERY are both used to get data on other servers.

I have grouped this with the VALUES keyword as EXEC/OPENQUERY are dealing with similar data.
I have also added USE into this dictionart as I felt it is cleaner than adding into dictionary keyword.other.DML.sql but let me know if you disagree.
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @egamma to be a potential reviewer

@msftclas
Copy link

Hi @gontadu, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@aeschli
Copy link
Contributor

aeschli commented Jun 27, 2016

@gontadu Thanks for the pull request, can you also make this request against https://github.com/textmate/sql.tmbundle/ ?
Thanks a lot!

@aeschli aeschli merged commit fef7e1c into microsoft:master Jun 27, 2016
@aeschli aeschli added this to the June 2016 milestone Jun 27, 2016
@gontadu
Copy link
Contributor Author

gontadu commented Jun 28, 2016

Ok will do now!

DONE: textmate/sql.tmbundle#15

@gontadu gontadu deleted the patch-1 branch July 22, 2016 21:37
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants