Merge pull request #23 from edx/clintonb/pylint-update

Upgraded to Pylint 1.5.x

edx_lint/files/pylintrc

@@ -7,9 +7,6 @@
 # pygtk.require().
 #init-hook=
 
-# Profiled execution.
-profile=no
-
 # Add files or directories to the blacklist. They should be base names, not
 # paths.
 ignore=
@@ -84,10 +81,6 @@ reports=no
 # (RP0004).
 evaluation=10.0 - ((float(5 * error + warning + refactor + convention) / statement) * 10)
 
-# Add a comment according to your evaluation note. This is used by the global
-# evaluation report (RP0004).
-comment=no
-
 
 # Template used to display messages. This is a python new-style format string
 # used to format the message information. See doc for all details
@@ -96,9 +89,6 @@ comment=no
 
 [BASIC]
 
-# Required attributes for module, separated by a comma
-required-attributes=
-
 # List of builtins function names that should not be used, separated by a comma
 bad-functions=map,filter,apply,input
 
@@ -205,10 +195,6 @@ ignore-mixin-members=yes
 # (useful for classes with attributes dynamically set).
 ignored-classes=SQLObject
 
-# When zope mode is activated, add a predefined set of Zope acquired attributes
-# to generated-members.
-zope=no
-
 # C Extension modules must be imported to be inspected.  For security reasons,
 # this # is not done by default, but we expect all imported code to be run at
 # some point anyway, so we explicitly allow it here
@@ -261,10 +247,6 @@ additional-builtins=
 
 [CLASSES]
 
-# List of interface methods to ignore, separated by a comma. This is used for
-# instance to not check methods defines in Zope's Interface base class.
-ignore-iface-methods=isImplementedBy,deferred,extends,names,namesAndDescriptions,queryDescriptionFor,getBases,getDescriptionFor,getDoc,getName,getTaggedValue,getTaggedValueTags,isEqualOrExtendedBy,setTaggedValue,isImplementedByInstancesOf,adaptWith,is_implemented_by
-
 # List of method names used to declare (i.e. assign) instance attributes.
 defining-attr-methods=__init__,__new__,setUp
 

pylintrc

@@ -58,7 +58,6 @@ evaluation = 10.0 - ((float(5 * error + warning + refactor + convention) / state
 comment = no
 
 [BASIC]
-required-attributes = 
 bad-functions = map,filter,apply,input
 module-rgx = (([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+))$
 const-rgx = (([A-Z_][A-Z0-9_]*)|(__.*__)|log|urlpatterns)$

setup.py

@@ -4,7 +4,7 @@ from setuptools import setup
 
 setup(
     name='edx-lint',
-    version='0.3.2',
+    version='0.4.0',
     description='edX-authored pylint checkers',
     url='https://github.com/edx/edx-lint',
     author='edX',
@@ -30,7 +30,7 @@ setup(
     },
 
     install_requires=[
-        'pylint==1.4.4',
-        'pylint-django==0.6.1',
+        'pylint>=1.5.1,<1.6.0',
+        'pylint-django>=0.7.1,<1.0.0',
     ],
 )

test/test_pylint_plugins.py

@@ -14,13 +14,6 @@ def load_tests(loader, tests, pattern):
     # Load our plugin.
     linter.load_plugin_modules(['edx_lint.pylint'])
 
-    # Configure the linter that runs the tests.
-
-    # This line prevents pylint from complaining about missing __revision__ in
-    # all the test files. But is this removing other required attributes that
-    # maybe we do want to check for?
-    linter.global_set_option('required-attributes', ())
-
     here = os.path.dirname(os.path.abspath(__file__))
 
     tests = make_tests(