-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support Python f-string #69
Comments
Added syntax highlighting for multi-line f-strings in 2.14 |
Sorry, but I don't understand how it works. I tried with your Python example but got this result: So, I conclude that doesn't work. You explain here never to do that. Ok, this is not a good practice. But should we understand that it is impossible with your lib ? Thanks for explanations |
No, it's not my place to "punish" people for something that might be unsafe. And there is no indication of unsanitized input here. I also prematurely closed it without adding support for single-line f-strings) Will add later. The only string that does not work by design (so far) is this one: multiline_without_marker = f"""
SELECT * FROM {table_name};
""" I've open separate issue for queries like this one: #78. Thanks for your input. |
As far as I can tell, single-line f-strings are still not supported :( |
It's impossible to parse Python f-strings for the moment:
✅
❌
It would be very practical to add this possibility 🙏
The text was updated successfully, but these errors were encountered: