Skip to content

Commit

Permalink
More CI fixes (#208)
Browse files Browse the repository at this point in the history
* Remove bandit from CI, update poetry.lock&pre-commit-config.yaml

* We don't support python 3.6

* poetry install also on tests flow

* remove pytest-azurepipelines
  • Loading branch information
rytilahti committed Sep 23, 2021
1 parent 202d107 commit d720288
Show file tree
Hide file tree
Showing 4 changed files with 126 additions and 118 deletions.
7 changes: 2 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,10 +53,6 @@ jobs:
- name: "Run check-ast"
run: |
poetry run pre-commit run check-ast --all-files
- name: "Potential security issues (bandit)"
run: |
poetry run pre-commit run bandit --all-files
tests:
name: "Python ${{ matrix.python-version}} on ${{ matrix.os }}"
Expand All @@ -65,7 +61,7 @@ jobs:

strategy:
matrix:
python-version: ["3.6", "3.7", "3.8", "3.9", "pypy3"]
python-version: ["3.7", "3.8", "3.9", "pypy3"]
os: [ubuntu-latest, macos-latest, windows-latest]

steps:
Expand All @@ -76,6 +72,7 @@ jobs:
- name: "Install dependencies"
run: |
python -m pip install --upgrade pip poetry
poetry install
- name: "Run tests"
run: |
poetry run pytest --cov kasa --cov-report xml
Expand Down
8 changes: 4 additions & 4 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,13 @@ repos:
- id: check-ast

- repo: https://github.com/asottile/pyupgrade
rev: v2.19.4
rev: v2.27.0
hooks:
- id: pyupgrade
args: ['--py36-plus']

- repo: https://github.com/python/black
rev: 21.6b0
rev: 21.9b0
hooks:
- id: black

Expand All @@ -27,13 +27,13 @@ repos:
additional_dependencies: [flake8-docstrings]

- repo: https://github.com/pre-commit/mirrors-isort
rev: v5.8.0
rev: v5.9.3
hooks:
- id: isort
additional_dependencies: [toml]

- repo: https://github.com/pre-commit/mirrors-mypy
rev: v0.902
rev: v0.910
hooks:
- id: mypy
additional_dependencies: [types-click]

0 comments on commit d720288

Please sign in to comment.