Skip to content

Commit

Permalink
fix: e2e main test node setup is failing because of missing pnpm
Browse files Browse the repository at this point in the history
This pr is moving pnpm install step before node setup step.

Signed-off-by: Denis Golovin <[email protected]>
  • Loading branch information
dgolovin committed Dec 13, 2024
1 parent 2b973da commit 12e40eb
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/e2e-main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,12 @@ jobs:
ref: main
path: podman-desktop

- uses: pnpm/action-setup@v4
name: Install pnpm
with:
run_install: false
package_json_file: ./podman-desktop/package.json

- uses: actions/setup-node@v4
with:
node-version: 20
Expand Down Expand Up @@ -91,12 +97,6 @@ jobs:
# allow unprivileged user namespace
sudo sysctl -w kernel.apparmor_restrict_unprivileged_userns=0
- uses: pnpm/action-setup@v4
name: Install pnpm
with:
run_install: false
package_json_file: ./podman-desktop/package.json

- name: Execute pnpm
working-directory: ./podman-desktop
run: pnpm install
Expand Down

0 comments on commit 12e40eb

Please sign in to comment.