Skip to content

Commit

Permalink
[WIP] Fix test-functional.yml and .github/actions/install-all-deps/ac…
Browse files Browse the repository at this point in the history
…tion.yml to call the setup actions in this branch
  • Loading branch information
whitphx committed Mar 29, 2024
1 parent de2f364 commit 571823b
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .github/actions/install-all-deps/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: 'install all deps'
description: 'Install all deps'

inputs:
always_install_pnpm:
always_install_pnpm:
description: 'Dictates whether or not we should install pnpm & dependencies, regardless of the cache'
default: 'false'
node_auth_token:
Expand Down Expand Up @@ -85,10 +85,10 @@ runs:
shell: bash
run: |
. ${{ env.VENV_ACTIVATE }}
python -m pip install -e client/python
python -m pip install -e client/python
python -m pip install -e ".[oauth]"
- name: install-frontend
uses: "gradio-app/gradio/.github/actions/install-frontend-deps@main"
uses: "./.github/actions/install-frontend-deps"
with:
always_install_pnpm: ${{ inputs.always_install_pnpm }}
node_auth_token: ${{ inputs.node_auth_token }}
Expand Down
5 changes: 4 additions & 1 deletion .github/workflows/test-functional.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@ on:
workflows: ["trigger"]
types:
- requested
pull_request:
branches:
- main

permissions:
statuses: write
Expand Down Expand Up @@ -49,7 +52,7 @@ jobs:
repository: ${{ needs.changes.outputs.mergeable == 'true' && github.repository || needs.changes.outputs.source_repo }}
- name: install dependencies
id: install_deps
uses: "gradio-app/gradio/.github/actions/install-all-deps@main"
uses: "./.github/actions/install-all-deps"
with:
always_install_pnpm: true
build_lite: true
Expand Down

0 comments on commit 571823b

Please sign in to comment.