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.

Sign up for GitHub

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

Jump to bottom

Missing `f` prefix on f-strings #19825

Closed

code-review-doctor opened this issue Apr 23, 2022 · 0 comments · Fixed by #19826

Contributor

code-review-doctor commented Apr 23, 2022

Some strings looks like they're meant to be f-strings but are missing the f prefix meaning variable interpolation won't happen.

superset/superset/db_engine_specs/presto.py

Line 1201 in 3ccfa56

msg = "Field [{k}] is not part of the portioning key"

I found this issue automatically. I'm a bot. Beep Boop 🦊. See other issues I found in your repo here

The text was updated successfully, but these errors were encountered:

code-review-doctor mentioned this issue

fix: Missing f prefix on f-strings #19826

Merged

etr2460 closed this as completed in #19826

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