PyCQA/pylint

Add configuration to promote/demote severity of messages #2293

nicktimko posted onGitHub

<!-- Hi there! Thank you for wanting to make pylint better. Before you submit this, make sure that this feature wasn't already requested or if it is not already implemented in the master branch. -->

Status Quo

Currently you can disable specific messages or severity classes (error, warning, refactor, convention), so to do pylint runs of various stringency, you could do one of

  • pylint -E ...
  • pylint --disable=R,C,W ....
  • pylint --disable=R,C ...

However, if you have a style guide that might mean one of the "convention" messages is a must-do, then you need to disable all the others, then you can bit-OR the exit code to fail a build on the refactor bit being set. Alternatively, parse the output (ick).

Desired feature

Add a configuration where you can map message codes/names to severity level.

For instance, if I say something like C0303=E or trailing-whitespace: error, and run pylint, even if there are no "true" error messages, a C0303 message would cause the exit code to have a 2 in it.

Help

I'd be willing to take a stab at a patch, but I'm also not sure if it could be done in a plugin just as well? I'm having some trouble figuring out how. The docs point me towards a learning cliff: "In general, a plugin is a module which should have a function register, which takes an instance of pylint.lint.PyLinter as input.", but even looking at the class spec it's not immediately obvious how to hook into it. Are there docs for that I'm not finding or simple (but non-trivial) plugins for example?


Seems like we could look at a third party plugin to achive our goal.

If anyone has another to do this, may be share it here ?

posted by LvffY about 2 years ago

Fund this Issue

$0.00
Funded
Only logged in users can fund an issue

Pull requests