Skip to content

Conversation

@gontadu
Copy link
Contributor

@gontadu gontadu commented Jun 27, 2016

MySQL uses # as comments along with the standard SQL '--' at the beginning. However, # is not standard SQL and therefore when using SQL Server/Oracle, etc. things marked with # are still shown as comment in the code even if they aren't comments. SQL Server specifically uses # ## to denote temporary tables.

Therefore, until someone figures out how to write a mysql exception to the syntax highlighting, I would suggest we suppress this code.

MySQL uses # as comments along with the standard SQL '--' at the beginning. However, # is not standard SQL and therefore when using SQL Server/Oracle, etc. things marked with # are still shown as comment in the code even if they aren't comments. SQL Server specifically uses # ## to denote temporary tables.

Therefore, until someone figures out how to write a mysql exception to the syntax highlighting, I would suggest we suppress this code.
@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;

@gontadu gontadu changed the title # ## Pound Hash signs should not be marked as comments # ## Pound Hash signs should not be marked as comments in SQL Jun 27, 2016
@dbaeumer dbaeumer added the languages-basic Basic language support issues label Jun 27, 2016
@aeschli
Copy link
Contributor

aeschli commented Jun 27, 2016

@gontadu Please file this request to https://github.com/textmate/sql.tmbundle first.
I'm not enough informed on sql and it's flavour to decide if it is a good idea to remove that rule.

@gontadu
Copy link
Contributor Author

gontadu commented Jun 28, 2016

It is done upstream, but seeing the activity on that bundle.. it might take a while.

@gontadu gontadu closed this Jul 26, 2016
@gontadu gontadu deleted the patch-2 branch July 26, 2016 17:07
@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

languages-basic Basic language support issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants