Support pylint output about violation-comments-to-stash-plugin HOT 10 CLOSED

Uh, too many expectations and I should actually read the error message before opening a ticket. Apparently this was caused by an invalid regex to file pattern...

from violation-comments-to-stash-plugin.

@tomasbjerre Although, it's a bit confusing that front page says:

while Flake8- and Pylint-formats are different and flake8 does not execute pylint internally.

If the tool really already supports pylint-output, I would suggest to change README as:

- Flake8 (Pep8, Mccabe, PyFlakes) / PyLint

And on configuration page, put:

FLAKE8 / PYLINT:    _______________________

...just to make it more evident?

from violation-comments-to-stash-plugin.

I can add a pylint parser if you have a sample pylint report? Command for generating it and its output.

from violation-comments-to-stash-plugin.

Alright, I added couple of gists (with pylint 1.6.4):

https://gist.github.com/tuukkamustonen/a9994bb4eededde6c3095eba922e73c3
https://gist.github.com/tuukkamustonen/78d38559176eea7cfb87d74072b4c86e

from violation-comments-to-stash-plugin.

I just released this. Should be in the Jenkins update center within a few hours.

from violation-comments-to-stash-plugin.

Thanks, I'll give it a spin tomorrow!

from violation-comments-to-stash-plugin.

It seems to work, but it be possible to include also the "numeric" rule ID into Rule field:

Found 1 violation:
Reporter: PYLINT
Rule: anomalous-backslash-in-string
Severity: WARN
File: fs/csm/admin_api/admin_api.py L1086

Anomalous backslash in string: '\s'. String constant might be missing an r prefix.
<this is a auto generated comment from violation-comments-lib F7F8ASD8123FSDF>

Normally you discuss/ignore/google per rule ID (e.g. E1101) rather than it's label. Perfect output would be something like:

Rule: W1401(anomalous-backslash-in-string)

from violation-comments-to-stash-plugin.

Should be fixed in 1.31!

from violation-comments-to-stash-plugin.

I just realized I missed the severity level... I'll fix that later.

from violation-comments-to-stash-plugin.

Open issue again if not fixed!

from violation-comments-to-stash-plugin.