search

Ericsson / codechecker

CodeChecker is an analyzer tooling, defect database and viewer extension for the Clang Static Analyzer and Clang Tidy
https://codechecker.readthedocs.io
Apache License 2.0
2.15k stars 357 forks source link

[cfg] Upgrade to pylint 3.2.4 #4279

Open bruntib opened 6 days ago

bruntib commented 3 hours ago

Thanks for the review, it's a great work at this PR :)

So, would you think, that we should rather allow the "elif" style? Is there any style checker that anyone doesn't agree with? The other common note was about f-strings, but I would say that for the logging library it's reasonable not to use them.

I would wait a little before I go through this change again, so it can be done only once. If no other suggestions arrive, I'll undo "elif" style.