Skip to content

Commit

Permalink
Install twine in cibuildwheel environment (#874)
Browse files Browse the repository at this point in the history
* Checkout full repo history in release workflow

* Fix double quoted string?

* Install twine as part of cibuildwheel

* Manually install twine

* Add python setup step

* Set up python using conda action

* Setup conda after cibuildwheel to avoid conflicts

* Correct minor typo

* Remove test code from workflow
  • Loading branch information
charlesbluca authored Oct 21, 2022
1 parent 010131e commit e9ff9cd
Showing 1 changed file with 18 additions and 14 deletions.
32 changes: 18 additions & 14 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,7 @@ on:
release:
types: [created]

# for testing
concurrency:
group: release-${{ github.head_ref }}
cancel-in-progress: true

# Required shell entrypoint to have properly activated conda environments
# Required shell entrypoint to have properly activated conda environments
defaults:
run:
shell: bash -l {0}
Expand All @@ -24,11 +19,8 @@ jobs:
python: ["8", "9", "10"] # 3.x
steps:
- uses: actions/checkout@v3
- name: Install dependencies
run: |
pip install setuptools setuptools-rust wheel twine
which python
pip list
with:
fetch-depth: 0
- name: Set up QEMU for linux aarch64
if: contains(matrix.os, 'ubuntu')
uses: docker/setup-qemu-action@v2
Expand Down Expand Up @@ -57,13 +49,25 @@ jobs:
package-dir: .
output-dir: dist
config-file: "dask_planner/pyproject.toml"
- name: Set up Python
uses: conda-incubator/setup-miniconda@v2
with:
miniforge-variant: Mambaforge
use-mamba: true
python-version: "3.8"
channel-priority: strict
- name: Build source distribution
if: contains(matrix.os, 'ubuntu') && matrix.python == '8'
run: |
pip install -U setuptools-rust
mamba install setuptools-rust
python setup.py sdist
- name: list dist files
run: ls -lh dist/
- name: Check dist files
run: |
mamba install twine
twine check dist/*
ls -lh dist/
- name: Upload binary wheels
uses: actions/upload-artifact@v3
with:
Expand Down

0 comments on commit e9ff9cd

Please sign in to comment.