Skip to content

Conversation

@shoriwe
Copy link
Contributor

@shoriwe shoriwe commented Dec 25, 2025

Patch for: #227

@shoriwe
Copy link
Contributor Author

shoriwe commented Dec 25, 2025

Here is the exact reference to the documentation:

A string constant is formed by enclosing the string in single quotes ('). A single quote within the string can be encoded by putting two single quotes in a row - as in Pascal. C-style escapes using the backslash character are not supported because they are not standard SQL.

@shoriwe shoriwe changed the title fix: sqlite single quote interpolations fix: sqlite: string interpolation incorrectly uses backslash escapes Dec 25, 2025
@shoriwe
Copy link
Contributor Author

shoriwe commented Dec 25, 2025

I just added support for the other cases where interpolation is incorrectly using backslash in SQLite

@huandu huandu added the bug label Dec 29, 2025
@huandu
Copy link
Owner

huandu commented Dec 29, 2025

Thanks for your contribution.

Copy link
Owner

@huandu huandu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor issues in comments.

@huandu
Copy link
Owner

huandu commented Dec 29, 2025

@shoriwe Don't forget to fix failed cases. Thanks.

@shoriwe
Copy link
Contributor Author

shoriwe commented Dec 29, 2025

@shoriwe Don't forget to fix failed cases. Thanks.

Made all SQLite test pass by resolving to valid SQLite string syntax

@coveralls
Copy link

coveralls commented Dec 29, 2025

Coverage Status

coverage: 96.999% (+0.02%) from 96.975%
when pulling 2994d27 on shoriwe:fix/sqlite-strings
into e4f129e on huandu:master.

@shoriwe
Copy link
Contributor Author

shoriwe commented Dec 29, 2025

@huandu. I have updated the PR, let me know if it is now ready to merge

@huandu huandu merged commit 21cf114 into huandu:master Dec 30, 2025
2 checks passed
@huandu
Copy link
Owner

huandu commented Dec 30, 2025

LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants