Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Prevent invalid SQL detection when keyword is part of something else#2

Open
Jenselme wants to merge 4 commits into
Submersible:mainSubmersible/better-python-string-sql:mainfrom
Jenselme:fix-wrong-detectionJenselme/better-python-strings:fix-wrong-detectionCopy head branch name to clipboard
Open

Prevent invalid SQL detection when keyword is part of something else#2
Jenselme wants to merge 4 commits into
Submersible:mainSubmersible/better-python-string-sql:mainfrom
Jenselme:fix-wrong-detectionJenselme/better-python-strings:fix-wrong-detectionCopy head branch name to clipboard

Conversation

@Jenselme
Copy link
Copy Markdown

This could happen for instance when a string contained attachedField: attach was incorrectly detected as the start of some SQL. This resulted in a random word being highlighted and the Python syntax highlighting to be messed up.

This could happen for instance when a string contained attachedField:
attach was incorrectly detected as the start of some SQL. This resulted
in a random word being highlighted and the Python syntax highlighting to
be messed up.
@RadoslawDeska
Copy link
Copy Markdown

RadoslawDeska commented Oct 30, 2024

This single quote part of regex causes improper highlighting of parentheses when 'somestring' is used inside """ """ code:
WITHOUT the single quote option:
obraz

WITH the single quote option:
obraz

@RadoslawDeska
Copy link
Copy Markdown

I forked the project to improve on the thing as well as in Python print()

https://github.com/RadoslawDeska/better-python-string-sql-refined/

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.