Merge pull request #4274 from edx/db/update-pylint

Update pylint to 1.4.0

pavelib/quality.py

@@ -42,8 +42,10 @@ def find_fixme(options):
 
         sh(
             "{pythonpath_prefix} pylint --disable R,C,W,E --enable=fixme "
-            "-f parseable {apps} | tee {report_dir}/pylint_fixme.report".format(
+            "--msg-template={msg_template} {apps} "
+            "| tee {report_dir}/pylint_fixme.report".format(
                 pythonpath_prefix=pythonpath_prefix,
+                msg_template='"{path}:{line}: [{msg_id}({symbol}), {obj}] {msg}"',
                 apps=apps_list,
                 report_dir=report_dir
             )
@@ -97,10 +99,11 @@ def run_pylint(options):
         )
 
         sh(
-            "{pythonpath_prefix} pylint {flags} -f parseable {apps} | "
+            "{pythonpath_prefix} pylint {flags} --msg-template={msg_template} {apps} | "
             "tee {report_dir}/pylint.report".format(
                 pythonpath_prefix=pythonpath_prefix,
                 flags=" ".join(flags),
+                msg_template='"{path}:{line}: [{msg_id}({symbol}), {obj}] {msg}"',
                 apps=apps_list,
                 report_dir=report_dir
             )

requirements/edx/base.txt

@@ -129,7 +129,7 @@ nose-exclude
 nose-ignore-docstring
 nosexcover==1.0.7
 pep8==1.5.7
-pylint==0.28
+pylint==1.4.0
 python-subunit==0.0.16
 rednose==0.3
 selenium==2.42.1

scripts/all-tests.sh

@@ -56,7 +56,7 @@ set -e
 ###############################################################################
 
 # Violations thresholds for failing the build
-PYLINT_THRESHOLD=4600
+PYLINT_THRESHOLD=5800
 PEP8_THRESHOLD=0
 
 source $HOME/jenkins_env