Skip to content

fix: sync pnpm lockfiles with package.json to unblock Terraform CI - #332

Merged
max-ostapenko merged 2 commits into
mainfrom
copilot/fix-github-actions-terraform
Jul 14, 2026
Merged

fix: sync pnpm lockfiles with package.json to unblock Terraform CI#332
max-ostapenko merged 2 commits into
mainfrom
copilot/fix-github-actions-terraform

Conversation

Copilot AI commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

The Terraform CI job was failing because Docker builds run pnpm install --prod --frozen-lockfile, which requires lockfile specifiers to exactly match package.json. The lockfiles had drifted behind the package.json versions.

Stale specifiers

Service Package Lockfile package.json
bigquery-export @google-cloud/firestore 8.0.0 8.6.0
dataform-service @google-cloud/dataform 2.2.1 2.4.0
dataform-service @google-cloud/run 3.0.1 3.3.0
dataform-service @google-cloud/storage 7.19.0 7.21.0

Fix

Regenerated pnpm-lock.yaml for both infra/bigquery-export/src/ and infra/dataform-service/src/ to align with current package.json declarations.

- bigquery-export: update @google-cloud/firestore 8.0.0 -> 8.6.0
- dataform-service: update @google-cloud/dataform 2.2.1 -> 2.4.0,
  @google-cloud/run 3.0.1 -> 3.3.0, @google-cloud/storage 7.19.0 -> 7.21.0

Fixes pnpm install --frozen-lockfile failure in Docker builds during
Terraform apply.
Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'Terraform' fix: sync pnpm lockfiles with package.json to unblock Terraform CI Jul 14, 2026
Copilot AI requested a review from max-ostapenko July 14, 2026 11:32
@max-ostapenko
max-ostapenko marked this pull request as ready for review July 14, 2026 12:02
@max-ostapenko
max-ostapenko merged commit fce86da into main Jul 14, 2026
34 checks passed
@max-ostapenko
max-ostapenko deleted the copilot/fix-github-actions-terraform branch July 14, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants