Skip to content

Commit 4dcb05a

Browse files
authored
Cye/fix test pypi publish (#947)
### Description <!-- Provide a detailed description of the changes in this PR --> - Fix TestPyPI publishing path in the sub-package workflow. ### Type of changes <!-- Mark the relevant option with an [x] --> - [x] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Refactor - [ ] Documentation update - [ ] Other (please describe): ### Usage <!--- How does a user interact with the changed code --> - Run this: https://github.com/NVIDIA/bionemo-framework/actions/workflows/bionemo-subpackage-ci.yml on this PR branch! ### Testing - https://github.com/NVIDIA/bionemo-framework/actions/runs/15692407014 ### Pre-submit Checklist <!--- Ensure all items are completed before submitting --> - [x] I have tested these changes locally - [ ] I have updated the documentation accordingly - [ ] I have added/updated tests as needed - [ ] All existing tests pass successfully --------- Signed-off-by: Cory Ye <[email protected]>
1 parent 98065b0 commit 4dcb05a

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/bionemo-subpackage-ci.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ jobs:
193193
name: Publish ${{ matrix.package }} to PyPI.
194194
runs-on: ubuntu-latest
195195
environment:
196-
name: ${{ github.event.inputs.pypi && 'pypi' || 'testpypi' }}
196+
name: ${{ github.event.inputs.pypi == 'true' && 'pypi' || 'testpypi' }}
197197
url: ${{ github.event.inputs.pypi && format('https://pypi.org/p/{0}', matrix.package) || format('https://test.pypi.org/p/{0}', matrix.package) }}
198198
permissions:
199199
id-token: write
@@ -205,7 +205,7 @@ jobs:
205205
name: ${{ matrix.package }}-build-artifacts
206206
path: sub-packages/${{ matrix.package }}/dist
207207
- id: publish-to-testpypi
208-
name: Publish distribution 📦 to Test PyPI for PR.
208+
name: Publish distribution 📦 to Test PyPI.
209209
if: ${{ github.event.inputs.pypi == 'false' }}
210210
uses: pypa/gh-action-pypi-publish@release/v1
211211
with:
@@ -214,7 +214,7 @@ jobs:
214214
repository-url: https://test.pypi.org/legacy/
215215
skip-existing: ${{ github.event.inputs.version_overwrite }}
216216
- id: publish-to-pypi
217-
name: Publish distribution 📦 to PyPI for Workflow Dispatch.
217+
name: Publish distribution 📦 to PyPI.
218218
# To require testing before publishing to PyPI, add: ... && needs.install-and-test.result == 'success'
219219
# If testing is run but fails, the workflow will fail and not publish to PyPI (or Test PyPI).
220220
# We strongly recommend testing when publishing to production PyPI.

0 commit comments

Comments
 (0)