Add pylint and mypy linting

.travis.yml

@@ -13,7 +13,7 @@ matrix:
     # and prints a coverage report at the end.
     - env: PYTHON_VERSION="3.6" COVERAGE="true"
     # This environment doesn't run tests, but does check for lint violations.
-    # - env: PYTHON_VERSION="3.6" RUN_LINT="true" SKIP_TESTS="true"
+    - env: PYTHON_VERSION="3.6" RUN_LINT="true" SKIP_TESTS="true"
 
 notifications:
   email: false

build_tools/travis/test_script.sh

@@ -16,7 +16,7 @@ run_tests() {
 
 if [[ "$RUN_LINT" == "true" ]]; then
     echo "Running linter and mypy"
-    pylint --disable-warnings kglm
+    pylint kglm
     mypy --ignore-missing-imports --no-strict-optional kglm
 fi
 

requirements.txt

@@ -1,5 +1,11 @@
 git+git://github.com/rloganiv/allennlp.git@pytorch-grad-clip
 
+# Linting
+# Checks style, syntax, and other useful errors.
+pylint==1.8.1
+# Static type checking
+mypy==0.521
+
 # Coverage reports
 pytest-cov
 codecov