Skip to content

Commit

Permalink
More fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
AlexanderWells-diamond committed Sep 17, 2024
1 parent c13766c commit 5b054c3
Showing 1 changed file with 3 additions and 29 deletions.
32 changes: 3 additions & 29 deletions .github/workflows/code.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,17 +53,13 @@ jobs:
strategy:
fail-fast: false
matrix:
os: [ubuntu-latest, windows-latest, macos-latest]
python: [cp37, cp38, cp39, cp310]
os: [ubuntu-latest, windows-latest, macos-13]
python: [cp37, cp38, cp39, cp310, cp311, cp312]

exclude:
# MacOS 14.4.1 for arm64 doesn't support Python < 3.8
- os: macos-latest
python: "cp37"

include:
# Put coverage and results files in the project directory for mac
- os: macos-latest
- os: macos-13
cov_file: "{project}/dist/coverage.xml"
results_file: "{project}/dist/pytest-results.xml"
# And for windows
Expand All @@ -74,11 +70,6 @@ jobs:
- os: ubuntu-latest
cov_file: /output/coverage.xml
results_file: /output/pytest-results.xml
# MacOS 13 required for Python < 3.8
- os: macos-13
python: "cp37"
cov_file: "{project}/dist/coverage.xml"
results_file: "{project}/dist/pytest-results.xml"

name: build/${{ matrix.os }}/${{ matrix.python }}
runs-on: ${{ matrix.os }}
Expand Down Expand Up @@ -126,23 +117,6 @@ jobs:
name: ${{ matrix.os }}/${{ matrix.python }}
directory: dist

publish-test-results:
name: Publish Unit Tests Results
needs: build
runs-on: ubuntu-latest
if: always()

steps:
- name: Download Artifacts
uses: actions/download-artifact@v4
with:
path: artifacts

- name: Publish Unit Test Results
uses: EnricoMi/publish-unit-test-result-action@v1
with:
files: artifacts/**/*.xml

test-sdist:
needs: [sdist]
strategy:
Expand Down

0 comments on commit 5b054c3

Please sign in to comment.