Pull out pylint to separate checker

jamesls · jamesls · commit 670992d4fdac · 2015-12-28T17:20:26.000-08:00
There's just too many false positives to be practical.
Rather than litter the code with pylint exceptions, I'd
prefer to just make "pylint -E" part of the travis builds
and have the fuller "make pylint" just a manual process we
occassionally run.
diff --git a/Makefile b/Makefile
@@ -18,19 +18,25 @@ check:
 	#
 	#
 	#
-	###### PYLINT ######
-	# Python linter.
+	# Proper docstring conventions according to pep257
 	#
 	#
+	pep257 --add-ignore=D100,D101,D102,D103,D104,D105,D204
 	#
-	pylint --rcfile .pylintrc awsshell
 	#
 	#
+	###### PYLINT ERRORS ONLY ######
 	#
-	# Proper docstring conventions according to pep257
 	#
 	#
-	pep257 --add-ignore=D100,D101,D102,D103,D104,D105
+	pylint --rcfile .pylintrc -E awsshell
 
+pylint:
+	###### PYLINT ######
+	# Python linter.  This will generally not have clean output.
+	# So you'll need to manually verify this output.
+	#
+	#
+	pylint --rcfile .pylintrc awsshell
 coverage:
 	py.test --cov awsshell --cov-report term-missing tests/

Original file line number	Diff line number	Diff line change
`@@ -18,19 +18,25 @@ check:`
`18`	`18`	`#`
`19`	`19`	`#`
`20`	`20`	`#`
`21`		`- ###### PYLINT ######`
`22`		`- # Python linter.`
	`21`	`+ # Proper docstring conventions according to pep257`
`23`	`22`	`#`
`24`	`23`	`#`
	`24`	`+ pep257 --add-ignore=D100,D101,D102,D103,D104,D105,D204`
`25`	`25`	`#`
`26`		`- pylint --rcfile .pylintrc awsshell`
`27`	`26`	`#`
`28`	`27`	`#`
	`28`	`+ ###### PYLINT ERRORS ONLY ######`
`29`	`29`	`#`
`30`		`- # Proper docstring conventions according to pep257`
`31`	`30`	`#`
`32`	`31`	`#`
`33`		`- pep257 --add-ignore=D100,D101,D102,D103,D104,D105`
	`32`	`+ pylint --rcfile .pylintrc -E awsshell`
`34`	`33`
	`34`	`+pylint:`
	`35`	`+ ###### PYLINT ######`
	`36`	`+ # Python linter. This will generally not have clean output.`
	`37`	`+ # So you'll need to manually verify this output.`
	`38`	`+ #`
	`39`	`+ #`
	`40`	`+ pylint --rcfile .pylintrc awsshell`
`35`	`41`	`coverage:`
`36`	`42`	`py.test --cov awsshell --cov-report term-missing tests/`