Skip to content

Commit

Permalink
cleanup working publish.yml and prepare for PR
Browse files Browse the repository at this point in the history
  • Loading branch information
jdegenstein authored Nov 30, 2023
1 parent 73382b6 commit db40a29
Showing 1 changed file with 5 additions and 21 deletions.
26 changes: 5 additions & 21 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
@@ -1,30 +1,15 @@
name: Wheel building and publishing

on: [push, pull_request, workflow_dispatch]
on: [push, pull_request, workflow_dispatch] # TODO: update this later

jobs:
# may be useful to have a wheel building test here eventually (untested):
# test_wheel_building:
# #
# permissions:
# contents: none
# uses: someactionyml.yml #TODO: replace with wheel build action
# if: (github.event_name == 'push') # || github.event_name == 'pull_request')
# with:
# upload_to_pypi: false
# targets: |
# - cp311-manylinux_x86_64

# secrets:
# pypi_token: ${{ secrets.pypi_token }} # consider using alternate strategy

build_wheel:
# This does the actual wheel building or if triggered manually via the workflow dispatch, or for a tag.
# this job does NOT publish the wheel
name: Build wheel on ubuntu-latest
runs-on: ubuntu-latest
if: github.event_name == 'workflow_dispatch'
#if: (github.repository == 'gumyr/build123d' && ( startsWith(github.ref, 'refs/tags/v') || github.event_name == 'schedule' || github.event_name == 'workflow_dispatch' || contains(github.event.pull_request.labels.*.name, 'Build wheel')))
#if: github.event_name == 'workflow_dispatch'
if: (github.repository == 'gumyr/build123d' && ( startsWith(github.ref, 'refs/tags/v'))) || github.event_name == 'workflow_dispatch'
steps:
- uses: actions/checkout@v4
with:
Expand All @@ -36,14 +21,13 @@ jobs:
pwd
ls -lR
python3 -m pip install --upgrade pip
pip -V
python3 -m pip -V
python3 -m pip freeze
python3 -m pip wheel -w wheelhouse .
python3 -m pip freeze
- uses: actions/upload-artifact@v3
with:
path: ./wheelhouse/*.whl
path: ./wheelhouse/build123d*.whl # only store the build123d wheel

# Do we need sdist wheel?
# build_sdist:
Expand Down

0 comments on commit db40a29

Please sign in to comment.