Skip to content

Commit

Permalink
Fix typo in serverless deploy yaml (#204)
Browse files Browse the repository at this point in the history
Summary:
The action in question is prerun not pre-run. This was breaking deploys to versions that were different than the one in the hosted pyenv.

Test Plan:
Deploy to 3.8 on 22.04, no more error abuot missing python 3.8
  • Loading branch information
gibsondan authored Oct 28, 2024
1 parent 62eb083 commit 16278a2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions github/serverless/dagster-plus-deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,12 +60,12 @@ jobs:
# First ensure the correct Python version is installed
- name: Set up Python ${{ env.PYTHON_VERSION }} for target
id: setup-python-version
if: steps.pre-run.outputs.result == 'pex-deploy'
if: steps.prerun.outputs.result == 'pex-deploy'
uses: actions/setup-python@v5
with:
python-version: ${{ env.PYTHON_VERSION }}
- name: Install setuptools
if: steps.pre-run.outputs.result == 'pex-deploy'
if: steps.prerun.outputs.result == 'pex-deploy'
run: ${{ steps.setup-python-version.outputs.python-path }} -m pip install setuptools
shell: bash

Expand Down

0 comments on commit 16278a2

Please sign in to comment.