Skip to content

Commit 7e313a3

Browse files
committed
fix(build): build py dists before deploy
The old opentrons deploy actions would build the distributables as well as deploy them; now that we're using the pypi-maintained deploy actions, we need to build them explicitly (we can't use the ones we already built because that happened in a different job aka vm).
1 parent 068dbc8 commit 7e313a3

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

.github/workflows/api-test-lint-deploy.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -214,6 +214,10 @@ jobs:
214214
- uses: './.github/actions/python/setup'
215215
with:
216216
project: 'api'
217+
- name: 'build api distributables'
218+
shell: bash
219+
run: make -C api sdist wheel
220+
217221
# creds and repository configuration for deploying python wheels
218222
- if: ${{ !env.OT_TAG }}
219223
name: 'upload to test pypi'

.github/workflows/shared-data-test-lint-deploy.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,10 @@ jobs:
184184
script: |
185185
const { buildComplexEnvVars, } = require(`${process.env.GITHUB_WORKSPACE}/.github/workflows/utils.js`)
186186
buildComplexEnvVars(core, context)
187+
- name: 'build shared-data wheel'
188+
shell: bash
189+
run: make -C shared-data dist-py
190+
187191
# creds and repository configuration for deploying python wheels
188192
- if: ${{ !env.OT_TAG }}
189193
name: 'upload to test pypi'

0 commit comments

Comments
 (0)