Multi-line queries only execute when GO is the only token on a newline
Updated all code references for using GO instead of semicolon
Bug fix that caused some multi-line statements to crash when batching multiple queries
Tests for valid multi-line completion
Disclaimer: I have not accounted for an edge cause where a block comment may end on the same line as GO. For example, the following statement should execute but does not for us:
select 1 /* hey some comment
and the comment keeps going */ go
I have decided to forgo this edge case in favor of waiting for our dependency, sqlparse, to fix a bug that strips newlines from a method I'm using to strip comments (tracked here). I'm open to a discussion on this @udeeshagautam @chlafreniere.
Closes #175.
Changes include:
GOis the only token on a newlineGOinstead of semicolonDisclaimer: I have not accounted for an edge cause where a block comment may end on the same line as
GO. For example, the following statement should execute but does not for us:I have decided to forgo this edge case in favor of waiting for our dependency, sqlparse, to fix a bug that strips newlines from a method I'm using to strip comments (tracked here). I'm open to a discussion on this @udeeshagautam @chlafreniere.