Skip to content

Commit

Permalink
Start tidying up
Browse files Browse the repository at this point in the history
- Don't get tags when building from sdist, everything should be there
  already

- Try glob-matching the sdist location now we have the right path

- ls fewer files
  • Loading branch information
ajjackson committed Dec 10, 2024
1 parent 0c78bba commit a2104bf
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions .github/workflows/build_upload_pypi_wheels.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,8 @@ jobs:
steps:
- name: Checkout project (for test files)
uses: actions/checkout@v4
with:
fetch-depth: 0 # Ensure tags are fetched for versioning
# with:
# fetch-depth: 0 # Ensure tags are fetched for versioning

- name: Delete source (to ensure we are using sdist only)
shell: bash -l {0}
Expand All @@ -71,12 +71,12 @@ jobs:
name: python-source-distribution
path: dist/

- name: Rename downloaded sdist
shell: bash -l {0}
run: |
ls -R dist
mv $(find dist/ -name 'euphonic*.tar.gz') dist/euphonic.tar.gz
ls -R dist
# - name: Rename downloaded sdist
# shell: bash -l {0}
# run: |
# ls -R dist
# mv $(find dist/ -name 'euphonic*.tar.gz') dist/euphonic.tar.gz
# ls -R dist

- name: Setup Python ${{ matrix.python-version }}
uses: actions/setup-python@v5
Expand Down Expand Up @@ -133,7 +133,7 @@ jobs:

with:
output-dir: wheelhouse
package-dir: "dist/euphonic.tar.gz"
package-dir: "dist/python-source-distribution/euphonic*.tar.gz"

- name: Upload wheels as build artifacts
uses: actions/upload-artifact@v4
Expand Down Expand Up @@ -223,7 +223,7 @@ jobs:
merge-multiple: true

- name: List Files
run: ls -R
run: ls -R dist/

# - name: Upload wheels to PyPI
# uses: pypa/gh-action-pypi-publish@release/v1

0 comments on commit a2104bf

Please sign in to comment.