pylint rules outdated

80 views
Skip to first unread message

clark...@gmail.com

unread,
Mar 22, 2018, 5:59:27 AM3/22/18
to SonarQube
Hi everyone!

We are setting up a python project with sonarqube 6.5 and we are importing our pylint report when running an analysis.
The problem is that our sonarqube installation only knows of 180 pylint rules (28 of them being deprecated), and that pylint 1.8.3 we are using has 295 messages.

Looking into sonar-python/sonar-python-plugin/src/main/resources/org/sonar/plugins/python/pylint/rules.xml we can clearly see only pylint 1.2.1 (4 years old!!) is supported by sonar-python.

<!-- Added between Pylint 0.25 and 1.2.1 -->
<rule>
 
<key>C0204</key>
 
<name><![CDATA[Metaclass class method first argument]]></name>
 
<configKey>C0204</configKey>
 
<description>
    <![CDATA[Used when a metaclass [...]

So my questions are:
 - has sonar any plan to update sonar-python to support all actual rules from pylint, and better maintain pylint rules in the future?
 - is sonar completely relying on the community to keep pylint rules up to date?
 - given the actual state, is it recommended to import pylint report and is pylint support simply being deprecated?

Thanks in advance for your feedback!
Gabriel

clark...@gmail.com

unread,
Mar 22, 2018, 7:40:39 AM3/22/18
to SonarQube
FYI, I just created https://github.com/SonarSource/sonar-python/pull/144 to update pylint rules to match 1.8.3's 

Pierre-Yves Nicolas

unread,
Mar 27, 2018, 8:24:44 AM3/27/18
to SonarQube
Hi Gabriel,

Thank you for raising this point.
We should indeed have updated the pylint rules in SonarPython a long time ago.

Thanks a lot for the pull request: I had a look at it and added some comments.

Pierre-Yves

clark...@gmail.com

unread,
Mar 29, 2018, 4:03:12 AM3/29/18
to SonarQube
Thanks Pierre-Yves, that will be much appreciated!

Gabriel

Pierre-Yves Nicolas

unread,
Apr 16, 2018, 4:22:37 AM4/16/18
to clark...@gmail.com, SonarQube
This update is tracked as SONARPY-280.
It will be part of SonarPython 1.10.
There's an open RFF for that.

Thanks!
Pierre-Yves

--
You received this message because you are subscribed to a topic in the Google Groups "SonarQube" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/sonarqube/ezXW03MzzYU/unsubscribe.
To unsubscribe from this group and all its topics, send an email to sonarqube+unsubscribe@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/sonarqube/a6ae762f-1f90-49c3-b293-09024a66ccce%40googlegroups.com.

For more options, visit https://groups.google.com/d/optout.

Reply all
Reply to author
Forward
0 new messages