Skip to content

maintenance: update pnpm version #665

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 5, 2024
Merged

Conversation

MH4GF
Copy link
Contributor

@MH4GF MH4GF commented May 29, 2024

Problem

Currently, when dependabot runs CI, the following error occurs:

Run pnpm install --frozen-lockfile
 WARN  Ignoring not compatible lockfile at /home/runner/work/graphql-codegen-typescript-validation-schema/graphql-codegen-typescript-validation-schema/pnpm-lock.yaml
 ERR_PNPM_NO_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is absent

Note that in CI environments this setting is true by default. If you still need to run install in such cases, use "pnpm install --no-frozen-lockfile"
Error: Process completed with exit code 1.

ref: https://github.com/Code-Hex/graphql-codegen-typescript-validation-schema/actions/runs/9271635994/job/25507513605

pnpm-lock.yaml seems to be incompatible, but that was because the version of pnpm running in CI is older.

Solution

  • Bump up the pnpm version to v9.
  • Manage the pnpm version in package.json
    • pnpm is available with corepack enable.
    • You can also update the pnpm version with dependabot if you do it this way.
  • Refer to it in CI.

@MH4GF MH4GF marked this pull request as ready for review May 29, 2024 14:50
@MH4GF MH4GF changed the title update pnpm version maintenance: update pnpm version May 29, 2024
@Code-Hex
Copy link
Owner

Code-Hex commented Jun 5, 2024

@MH4GF Thanks! I fix this in another PR 🙏

@Code-Hex Code-Hex merged commit cca42dc into Code-Hex:main Jun 5, 2024
0 of 2 checks passed
@MH4GF MH4GF deleted the update-pnpm-version branch June 5, 2024 06:42
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