Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Missing f prefix on f-strings #5628

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

Missing f prefix on f-strings #5628

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

Comments

@code-review-doctor
Copy link
Contributor

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

raise ValueError("Got average f{average}, expected one of None, 'token', or 'batch'")

"Required parameters should be documented like: '{ident} : `{type}`'\n"

"Optional parameters should be documented like: '{ident} : `{type}`, optional (default = `{expr}`)'\n"

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant