Skip to content
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

Update pnpm version to v9.13.2 #529

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

damien-schneider
Copy link
Contributor

This pull request includes updates to the package.json file to improve compatibility and maintain up-to-date dependencies.

Dependency updates:

Copy link

changeset-bot bot commented Nov 17, 2024

⚠️ No Changeset found

Latest commit: 23042c6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Nov 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
supabase-cache-helpers ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 11:53pm

Copy link

vercel bot commented Nov 17, 2024

@damien-schneider is attempting to deploy a commit to the psteinroe's projects Team on Vercel.

A member of the Team first needs to authorize it.

@psteinroe
Copy link
Owner

@damien-schneider can you also adapt the CI workflow?

@damien-schneider
Copy link
Contributor Author

@damien-schneider can you also adapt the CI workflow?

You're right, it is done. Upgraded pnpm action-setup to v4 and pnpm to v9.x.x

@damien-schneider
Copy link
Contributor Author

damien-schneider commented Nov 18, 2024

@psteinroe Would you prefer to specify the exact version v9.13.2 throughout, or should we remove this version specification from the global package.json to allow flexibility?
Because if we want to keep an exact version, I should add this exact version to the CI too

Copy link

vercel bot commented Nov 21, 2024

Deployment failed with the following error:

Resource is limited - try again in 5 hours (more than 100, code: "api-deployments-free-per-day").

Copy link

codecov bot commented Nov 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.07%. Comparing base (8ed14cd) to head (23042c6).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #529   +/-   ##
=======================================
  Coverage   93.07%   93.07%           
=======================================
  Files         135      135           
  Lines        1992     1992           
  Branches      614      615    +1     
=======================================
  Hits         1854     1854           
  Misses        114      114           
  Partials       24       24           
Flag Coverage Δ
postgrest-core 93.07% <ø> (ø)
postgrest-react-query 93.07% <ø> (ø)
postgrest-server 93.07% <ø> (ø)
postgrest-swr 93.07% <ø> (ø)
storage-core 93.07% <ø> (ø)
storage-react-query 93.07% <ø> (ø)
storage-swr 93.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

Copy link

vercel bot commented Nov 21, 2024

Deployment failed with the following error:

Resource is limited - try again in 59 minutes (more than 100, code: "api-deployments-free-per-day").

@damien-schneider
Copy link
Contributor Author

Please let me know when you’re ready to merge. Keeping the branch updated and resolving pnpm-lock conflicts for each package update is a bit tedious. 😅

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