-
-
Notifications
You must be signed in to change notification settings - Fork 534
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
Activate pyflakes check #530
Conversation
Pull Request Test Coverage Report for Build 4359318064
💛 - Coveralls |
@@ -1796,16 +1794,16 @@ def _load_linear_scale(self, compu_scale, decimal): | |||
if lower_limit is not None and \ | |||
upper_limit is not None and \ | |||
lower_limit > upper_limit: | |||
LOGGER.warning(f'An valid interval should be provided for ' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we disable F541
? I'd like to be able to use F-strings even in situations where nothing is formatted...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's weird but it should be possible. Are you sure? 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure?
quite: IMO, having to use two different kinds of strings for things like
print(f"there is a problem ({id=}) with whatever stupid thing I wanted to do "
f"and this needs to be fixed before we can proceed because if not, "
f"world #{world_id} is gonna end")
looks pretty weird...
(also, I like to use f-strings for stuff which could potentially include formatting even if they do not at the moment.)
I'm not hell-bent on that matter, though, so if you really want F541, let's keep it active...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have a strong opinion about that one. I'll make the change and force push.
going in. thanks! |
No description provided.