Skip to content

Commit f7779b9

Browse files
authored
Merge pull request #882 from fractal-analytics-platform/update-coverage-py
Fix usage of coverage in GHA and bump version
2 parents ee900a7 + 1c1000a commit f7779b9

File tree

4 files changed

+146
-130
lines changed

4 files changed

+146
-130
lines changed

.github/workflows/ci_poetry.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,12 +124,14 @@ jobs:
124124
steps:
125125
- uses: actions/checkout@v4
126126

127+
- run: pipx install poetry==1.8.2
128+
- run: poetry config virtualenvs.in-project true
127129
- uses: actions/setup-python@v5
128130
with:
129131
python-version: "3.10"
130132

131133
- name: Install dependencies
132-
run: python -m pip install --upgrade coverage[toml]
134+
run: poetry install --only dev
133135

134136
- name: Download data
135137
uses: actions/download-artifact@v4
@@ -143,7 +145,7 @@ jobs:
143145
# We added this step to replace the variable `MERGE_COVERAGE_FILES: true`
144146
# in the next step, which had started to raise errors
145147
# (https://github.com/fractal-analytics-platform/fractal-server/pull/1725).
146-
run: coverage combine coverage-data-*
148+
run: poetry run coverage combine coverage-data-*
147149

148150
- name: Add coverage comment to Pull Requests
149151
id: coverage_comment

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
# Unreleased
44

55
* Tests:
6+
* Use locked version of `coverage` in GitHub action (\#882).
7+
* Bump `coverage` version from 6.5 to 7.6 (\#882).
68
* Skip `test_import_ome_zarr_image_BIA` (\#879).
79

810
# 1.3.4

0 commit comments

Comments
 (0)