Skip to content

Commit

Permalink
ci: fixes
Browse files Browse the repository at this point in the history
- update action versions
- test on more modern pythons
- unhook & bump poetry required version
- relax coverage goal for buggy statement counting
  • Loading branch information
ctz committed Oct 26, 2024
1 parent 2ff24bb commit bf88591
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 9 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,13 @@ jobs:
strategy:
matrix:
os: [ubuntu, windows, macos]
python: ['3.8', '3.9', '3.10']
python: ['3.8', '3.9', '3.10', '3.11', '3.12', '3.13']
steps:
- name: Check out code from GitHub
uses: actions/checkout@v2.3.4
uses: actions/checkout@v4
- name: Set up Python ${{ matrix.python }}
id: python
uses: actions/setup-python@v2.2.2
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python }}
- name: Install workflow dependencies
Expand All @@ -30,10 +30,10 @@ jobs:
run: poetry run black --check hpke tests
- name: Check types
run: poetry run mypy hpke
- name: Run tests and ensure 100% line coverage
run: poetry run pytest --cov-fail-under=100 --cov hpke tests
- name: Run tests and ensure ~100% line coverage
run: poetry run pytest --cov-fail-under=99.5 --cov hpke tests
- name: Upload coverage artifact
uses: actions/upload-artifact@v2.2.4
uses: actions/upload-artifact@v4
with:
name: coverage-${{ matrix.python }}-${{ matrix.os }}
path: .coverage
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
pip==21.2.4
poetry==1.1.8
pip==24.2
poetry==1.8.2

2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -20,5 +20,5 @@ black = "^22.3.0"
mypy = "^0.942"

[build-system]
requires = ["poetry-core>=1.0.0"]
requires = ["poetry-core"]
build-backend = "poetry.core.masonry.api"

0 comments on commit bf88591

Please sign in to comment.