Fix 3.10-ism that crept in and test 3.9 in CI #104
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: CI | |
| on: | |
| pull_request: | |
| push: | |
| branches: | |
| - main | |
| # The goal here is to cancel older workflows when a PR is updated (because it's pointless work) | |
| concurrency: | |
| group: ${{ github.workflow }}-${{ github.head_ref || github.ref_name }} | |
| cancel-in-progress: true | |
| jobs: | |
| unittest: | |
| name: unit tests | |
| runs-on: ubuntu-latest | |
| strategy: | |
| matrix: | |
| python-version: ["3.9", "3.10", "3.11", "3.12"] | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - name: Set up Python ${{ matrix.python-version }} | |
| uses: actions/setup-python@v5 | |
| with: | |
| python-version: ${{ matrix.python-version }} | |
| - name: Install dependencies | |
| run: | | |
| python -m pip install --upgrade pip | |
| pip install .[tests] | |
| - name: Test with pytest | |
| run: | | |
| python -m pytest --cov=chart_review --cov-report=xml | |
| - name: Check coverage report | |
| if: github.ref != 'refs/heads/main' | |
| uses: orgoro/[email protected] | |
| with: | |
| coverageFile: coverage.xml | |
| token: ${{ secrets.GITHUB_TOKEN }} | |
| thresholdAll: .99 | |
| thresholdNew: 1 | |
| thresholdModified: 1 | |
| lint: | |
| runs-on: ubuntu-latest | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - name: Install linters | |
| # black is synced with the .pre-commit-hooks version | |
| run: | | |
| python -m pip install --upgrade pip | |
| pip install .[dev] | |
| - name: Run pycodestyle | |
| # E203: pycodestyle is a little too rigid about slices & whitespace | |
| # See https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#slices | |
| # W503: a default ignore that we are restoring | |
| run: | | |
| pycodestyle --max-line-length=100 --ignore=E203,W503 . | |
| - name: Run bandit | |
| if: success() || failure() # still run bandit if above checks fail | |
| run: | | |
| bandit -c pyproject.toml -r . | |
| - name: Run black | |
| if: success() || failure() # still run black if above checks fails | |
| run: | | |
| black --check --verbose . |