Add pylint (#36)

Addressing pylint warnings in few dozens of commits.


Co-authored-by: mike0sv <mike0sv@gmail.com>

Author: aguschin

.github/workflows/check-test-release.yml

@@ -3,7 +3,7 @@ on:
   push:
   pull_request:
   schedule:
-  - cron: '0 9 * * 1'  # M H d m w (Mondays at 9:00)
+  - cron: '0 7 * * 1'  # M H d m w (Mondays at 7:00)
 jobs:
   check:
     if: startsWith(github.ref, 'refs/tags') || github.event_name == 'pull_request' || github.event_name == 'schedule' || github.repository_owner != 'iterative'
@@ -21,7 +21,7 @@ jobs:
         path: ~/.cache/pre-commit
         key: pre-commit|${{ env.PYSHA }}|${{ hashFiles('.pre-commit-config.yaml') }}
     - run: pip install -U pre-commit tox
-    - run: pre-commit run -a --show-diff-on-failure
+    - run: SKIP=pylint pre-commit run -a --show-diff-on-failure
   test:
     if: startsWith(github.ref, 'refs/tags') || github.event_name == 'pull_request' || github.event_name == 'schedule' || github.repository_owner != 'iterative'
     name: Test py${{ matrix.python }}
@@ -35,11 +35,12 @@ jobs:
         fetch-depth: 0
     - uses: actions/setup-python@v2
     # legacy-resolver makes install faster by order of magnitude
-    - run: pip install -U .[tests] --use-deprecated=legacy-resolver
+    - run: pip install -U pre-commit .[tests] --use-deprecated=legacy-resolver
     - run: pytest
       env:
         GITHUB_USERNAME: ${{ secrets.GH_USERNAME }}
         GITHUB_TOKEN: ${{ secrets.GH_TOKEN }}
+    - run: pre-commit run pylint -a --show-diff-on-failure
     - name: "Upload coverage to Codecov"
       uses: codecov/codecov-action@v1
       with:

.pre-commit-config.yaml

@@ -45,6 +45,13 @@ repos:
           - types-six
           - types-PyYAML
           - pydantic
+  - repo: local
+    hooks:
+      - id: pylint
+        name: pylint
+        entry: pylint
+        language: system
+        types: [ python ]
   # - repo: https://github.com/PyCQA/bandit
   #   rev: '1.7.0'
   #   hooks: