Skip to content

Commit

Permalink
fix: don't rely on makefile for python publishing
Browse files Browse the repository at this point in the history
  • Loading branch information
TristanSpeakEasy committed Jun 18, 2024
1 parent 7b71b81 commit 93ba90f
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 10 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/sdk-publish.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -134,16 +134,16 @@ jobs:
id: check-publish
run: |
if [ -f scripts/publish.sh ]; then
echo "publish_with_makefile=true" >> $GITHUB_OUTPUT
echo "publish_with_script=true" >> $GITHUB_OUTPUT
fi
- name: Publish with Makefile
if: steps.check-publish.outputs.publish_with_makefile == 'true'
- name: Publish with script
if: steps.check-publish.outputs.publish_with_script == 'true'
env:
PYPI_TOKEN: ${{ secrets.pypi_token }}
run: |
make publish
./scripts/publish.sh
- name: Legacy publish
if: steps.check-publish.outputs.publish_with_makefile != 'true'
if: steps.check-publish.outputs.publish_with_script != 'true'
env:
TWINE_USERNAME: __token__
TWINE_PASSWORD: ${{ secrets.pypi_token }}
Expand Down
10 changes: 5 additions & 5 deletions .github/workflows/workflow-executor.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -205,16 +205,16 @@ jobs:
id: check-publish
run: |
if [ -f scripts/publish.sh ]; then
echo "publish_with_makefile=true" >> $GITHUB_OUTPUT
echo "publish_with_script=true" >> $GITHUB_OUTPUT
fi
- name: Publish with Makefile
if: steps.check-publish.outputs.publish_with_makefile == 'true'
- name: Publish with script
if: steps.check-publish.outputs.publish_with_script == 'true'
env:
PYPI_TOKEN: ${{ secrets.pypi_token }}
run: |
make publish
./scripts/publish.sh
- name: Legacy publish
if: steps.check-publish.outputs.publish_with_makefile != 'true'
if: steps.check-publish.outputs.publish_with_script != 'true'
env:
TWINE_USERNAME: __token__
TWINE_PASSWORD: ${{ secrets.pypi_token }}
Expand Down

0 comments on commit 93ba90f

Please sign in to comment.