Skip to content

Commit

Permalink
further distinguish artifacts using SHA
Browse files Browse the repository at this point in the history
  • Loading branch information
berquist committed Feb 23, 2024
1 parent 26fb78f commit faf2051
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
---
# yamllint disable rule:line-length
name: test

# yamllint disable-line rule:truthy
Expand Down Expand Up @@ -65,15 +66,15 @@ jobs:
python -m pip install build twine
- name: Build distribution packages (binary wheel and source tarball)
run: |
python -m build --outdir ./dist/${{ matrix.python-version }}
python -m build --outdir ./dist/${{ github.sha }}/${{ matrix.python-version }}
- name: Check packages with twine
run: |
python -m twine check ./dist/${{ matrix.python-version }}/*
python -m twine check ./dist/${{ github.sha }}/${{ matrix.python-version }}/*
- name: Store the distribution packages
uses: actions/upload-artifact@v4
with:
name: python-package-distributions
path: ./dist/${{ matrix.python-version }}
path: ./dist/${{ github.sha }}/${{ matrix.python-version }}

check-installation:
name: Check installation of distribution 📦
Expand All @@ -96,15 +97,15 @@ jobs:
uses: actions/download-artifact@v4
with:
name: python-package-distributions
path: ./dist/${{ matrix.python-version }}
path: ./dist/${{ github.sha }}/${{ matrix.python-version }}
- name: Update environment
run: |
python -m pip install -U setuptools pip
- name: Install from wheel
run: |
python -m pip install ./dist/${{ matrix.python-version }}/*.whl
python -m pip install ./dist/${{ github.sha }}/${{ matrix.python-version }}/*.whl
if: matrix.install-method == 'wheel'
- name: Install from source
run: |
python -m pip install ./dist/${{ matrix.python-version }}/*.tar.gz
python -m pip install ./dist/${{ github.sha }}/${{ matrix.python-version }}/*.tar.gz
if: matrix.install-method == 'source'

0 comments on commit faf2051

Please sign in to comment.