🚚 move pylint to pre-commit (#4242)

Author: MarcoGorelli

.pre-commit-config.yaml

@@ -22,6 +22,12 @@ repos:
   rev: 20.8b1
   hooks:
     - id: black
+- repo: https://github.com/PyCQA/pylint
+  rev: pylint-2.6.0
+  hooks:
+    - id: pylint
+      args: [--rcfile=.pylintrc]
+      files: ^pymc3/
 - repo: local
   hooks:
     - id: watermark

.pylintrc

@@ -34,8 +34,7 @@ disable=all
 # Enable the message, report, category or checker with the given id(s). You can
 # either give multiple identifier separated by comma (,) or put this option
 # multiple time. See also the "--disable" option for examples.
-enable=import-error,
-       import-self,
+enable=import-self,
        reimported,
        wildcard-import,
        misplaced-future,

pymc3/backends/base.py

@@ -18,14 +18,14 @@
 """
 import itertools as itl
 import logging
-from typing import Dict, List, Optional
+from typing import List
 from abc import ABC
 
 import numpy as np
 import warnings
 import theano.tensor as tt
 
-from ..model import modelcontext, Model
+from ..model import modelcontext
 from .report import SamplerReport, merge_reports
 from ..util import get_var_name
 

scripts/lint.sh

@@ -2,9 +2,5 @@
 
 set -e
 
-if [[ "$RUN_PYLINT" == "true" ]]; then
-    . ./scripts/lint.sh
-fi
-
 _FLOATX=${FLOATX:=float64}
 THEANO_FLAGS="floatX=${_FLOATX},gcc.cxxflags='-march=core2'" pytest -v --cov=pymc3 --cov-report=xml "$@" --cov-report term