Skip to content

Commit c5d7e74

Browse files
committed
ci: fix GHA to not require host pnpm; run all steps via container exec
- Replace pnpm host invocations with scripts/repro/exec.sh calls - Avoid recursive package scripts inside container (call turbo/prettier directly)
1 parent 8365588 commit c5d7e74

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

.github/workflows/pr-check.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -45,23 +45,23 @@ jobs:
4545
${{ runner.os }}-pnpm-store-
4646
4747
- name: Mark repo as safe for git inside container
48-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm shell git config --global --add safe.directory /workspace
48+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh git config --global --add safe.directory /workspace
4949

5050
- name: Install deps (container)
51-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm ci:install
51+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm install --frozen-lockfile
5252

5353
- name: Disable turbo telemetry (container)
54-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm turbo:telemetry:disable
54+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm turbo telemetry disable
5555

5656
- name: Check formatting (container)
57-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm format:check
57+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm prettier . --check
5858

5959
- name: Build (container)
60-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm build -- --force
60+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm turbo build --force
6161

6262
- name: Test (container)
63-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm test
63+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm turbo test
6464

6565
- name: Require changeset to be present in PR (container)
6666
if: github.event.pull_request.user.login != 'dependabot[bot]'
67-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm shell pnpm changeset status --since origin/main
67+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm changeset status --since origin/main

.github/workflows/release.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ jobs:
5050
${{ runner.os }}-pnpm-store-
5151
5252
- name: Install deps (container)
53-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm ci:install
53+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm install --frozen-lockfile
5454

5555
- name: Lingo.dev (container)
5656
if: ${{ !inputs.skip_lingo }}
@@ -69,13 +69,13 @@ jobs:
6969
--parallel true
7070
7171
- name: Disable turbo telemetry (container)
72-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm turbo:telemetry:disable
72+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm turbo telemetry disable
7373

7474
- name: Build (container)
75-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm build -- --force
75+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm turbo build --force
7676

7777
- name: Test (container)
78-
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" pnpm test
78+
run: REPRO_PNPM_STORE="${{ env.REPRO_PNPM_STORE }}" bash scripts/repro/exec.sh pnpm turbo test
7979

8080
- name: Create Release Pull Request or Publish to npm
8181
id: changesets

0 commit comments

Comments
 (0)