Python-CI.yml

name: Python 🐍 Linting and run tests

on: [push, pull_request]

jobs:
  build:
    # We want to run on external PRs, but not on our own internal PRs as they'll be run
    # by the push to the branch. Without this if check, checks are duplicated since
    # internal PRs match both the push and pull_request events.
    if:
      github.event_name == 'push' || github.event.pull_request.head.repo.full_name != github.repository

    runs-on: ubuntu-latest
    strategy:
      matrix:
        python-version: [3.6, 3.7, 3.8, 3.9]

    steps:
      - uses: actions/checkout@v2

      - name: Set up Python ${{ matrix.python-version }}
        uses: actions/setup-python@v2
        with:
          python-version: ${{ matrix.python-version }}

      - name: Install dependencies
        run: |
          python -m pip install --upgrade pip
          pip install flake8 pytest coverage
          if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
          python -m pip install -e '.[d]'

      - name: Lint with flake8
        run: |
          # stop the build if there are Python syntax errors or undefined names
          flake8 okconfig --count --select=E9,F63,F7,F82 --show-source --statistics
          # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
          flake8 okconfig --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics

      - name: Check PEP8 compliance
        run: pycodestyle --ignore=W504,E501 okconfig/
        continue-on-error: true
        #if: ${{ false }} # Disable this step

      - name: Run tests
        run: coverage run -m pytest

      - name: Show coverage report
        run: coverage report --skip-covered --omit *tests*
        continue-on-error: true