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
Make bugbear slightly more polite #298
Conversation
Bugbear is great at catching bugs in our code, but can come across a bit rude when commenting on pull requests 😂 Obviously it's a robot, but we like to encourage polite discourse
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.
Seems fair. I’d like others thoughts if anyone comes by …
@@ -1192,8 +1192,8 @@ def visit_Lambda(self, node): | |||
} | |||
B015 = Error( | |||
message=( | |||
"B015 Pointless comparison. This comparison does nothing but waste " | |||
"CPU instructions. Either prepend `assert` or remove it." |
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 personally feel the blunt nature of the message makes it clearer of what the check looks for.
The new message will make people think into it more and possibly over complicate their actions. But open to others thoughts …
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 definitely see the tradeoff here, and agree that the cause of the error should be the first thing the user sees. The aim with the change of wording was to retain that, while softening the order to a suggestion.
Co-authored-by: Cooper Lees <me@cooperlees.com>
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.
Thanks! Sorry about being slow. I’m on holidays at the moment.
@cooperlees no probs! Thanks for accepting the change, hope your holidays were excellent |
Bugbear is great at catching bugs in our code, but can come across a bit rude when commenting on pull requests 😂 Obviously it's a robot, but we like to encourage polite discourse