Merge pull request #490 from sentinel-hub/develop

Release version 1.3.0

Author: zigaLuksic

.flake8

@@ -1,8 +1,9 @@
 [flake8]
-ignore = E203, W503
+ignore = E203, W503, C408
 exclude = .git, __pycache__, build, dist
 max-line-length= 120
 max-complexity = 15
+min_python_version = 3.7.0
 per-file-ignores =
     # imported but unused
     __init__.py: F401

.github/workflows/ci_action.yml

@@ -4,60 +4,76 @@ on:
   pull_request:
   push:
     branches:
-      - 'master'
-      - 'develop'
+      - "master"
+      - "develop"
   schedule:
-    - cron: '0 0 * * *'
+    - cron: "0 0 * * *"
 
 env:
   # The only way to simulate if-else statement
   CHECKOUT_BRANCH: ${{ github.event_name == 'schedule' && 'develop' || github.ref }}
 
-
 jobs:
-
-  check-code-black-isort-flake8:
+  check-pre-commit-hooks:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout branch
         uses: actions/checkout@v2
         with:
           ref: ${{ env.CHECKOUT_BRANCH }}
-    
+
       - name: Setup Python
         uses: actions/setup-python@v2
         with:
           python-version: "3.8"
           architecture: x64
 
-      - name: Prepare linters
-        run: pip install black[jupyter] isort flake8 nbqa
+      - name: Prepare pre-commit validators
+        run: |
+          pip install pre-commit
 
-      - name: Check code compliance with black
-        run: black . --check --diff
+      - name: Check code compliance with pre-commit validators
+        run: pre-commit run --all-files
 
-      - name: Check code compliance with isort
-        run: |
-          isort . --check --diff
-          nbqa isort . --nbqa-diff
+  check-code-pylint-and-mypy:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout branch
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ env.CHECKOUT_BRANCH }}
+
+      - name: Setup Python
+        uses: actions/setup-python@v2
+        with:
+          python-version: "3.8"
+          architecture: x64
 
-      - name: Check code compliance with flake8
+      - name: Install packages
         run: |
-          flake8 .
-          nbqa flake8 . --nbqa-exclude=examples/core/CoreOverview.ipynb
+          sudo apt-get update
+          sudo apt-get install -y build-essential libgdal-dev graphviz proj-bin gcc libproj-dev libspatialindex-dev
+          pip install -r requirements-dev.txt --upgrade
+          python install_all.py -e
+
+      - name: Run pylint
+        run: make pylint
+
+      - name: Run mypy
+        run: mypy core/eolearn/core
 
   test-on-github:
     runs-on: ubuntu-latest
     strategy:
       matrix:
         python-version:
-          - '3.7'
-          - '3.9'
-          - '3.10'
-        include: 
+          - "3.7"
+          - "3.9"
+          - "3.10"
+        include:
           # A flag marks whether full or partial tests should be run
           # We don't run integration tests on pull requests from outside repos, because they don't have secrets
-          - python-version: '3.8'
+          - python-version: "3.8"
             full_test_suite: ${{ github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository }}
     steps:
       - name: Checkout branch
@@ -86,10 +102,9 @@ jobs:
             --sh_client_secret "${{ secrets.SH_CLIENT_SECRET }}"
           pytest --cov --cov-report=term --cov-report=xml
 
-      - name: Run pylint and reduced tests
+      - name: Run reduced tests
         if: ${{ !matrix.full_test_suite }}
         run: |
-          make pylint
           pytest -m "not sh_integration"
 
       - name: Upload code coverage