fix lockfile - Githubissues

jnsdls commented 3 days ago

PR-Codex overview

The focus of this PR is to update dependencies, add pnpm to engines, and adjust lockfile settings for compatibility.

Detailed summary

Added pnpm to engines
Updated typescript to 5.5.2
Adjusted lockfile settings for compatibility

The following files were skipped due to too many changes: pnpm-lock.yaml

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

vercel[bot] commented 3 days ago

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

Name	Status	Preview	Comments	Updated (UTC)
thirdweb_playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 2, 2024 5:04am
thirdweb-www	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 2, 2024 5:04am
wallet-ui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 2, 2024 5:04am

changeset-bot[bot] commented 3 days ago

⚠️ No Changeset found

Latest commit: 373ce19286b37474112a232690e5c39dc0495988

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

graphite-app[bot] commented 3 days ago

Your org requires the Graphite merge queue for merging into main

Add the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

jnsdls commented 3 days ago

#3572 👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jnsdls and the rest of your teammates on Graphite

codspeed-hq[bot] commented 3 days ago

CodSpeed Performance Report

Merging #3572 will not alter performance

_{Comparing 07-01-fix_lockfile (373ce19) with main (1a2e056)}

Summary

✅ 9 untouched benchmarks

github-actions[bot] commented 3 days ago

size-limit report 📦

Path	Size	Loading time (3g)	Running time (snapdragon)	Total time
thirdweb (esm)	40.47 KB (-0.04% 🔽)	810 ms (-0.04% 🔽)	2 s (-39.17% 🔽)	2.8 s
thirdweb (cjs)	89.99 KB (-0.54% 🔽)	1.8 s (-0.54% 🔽)	2.6 s (-70.06% 🔽)	4.4 s
thirdweb (minimal + tree-shaking)	4.79 KB (0%)	96 ms (0%)	64 ms (-72.54% 🔽)	160 ms
thirdweb/chains (tree-shaking)	423 B (0%)	10 ms (0%)	55 ms (-26% 🔽)	65 ms
thirdweb/react (minimal + tree-shaking)	13.55 KB (+0.21% 🔺)	271 ms (+0.21% 🔺)	354 ms (-37.44% 🔽)	625 ms

codecov[bot] commented 3 days ago

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 62.35%. Comparing base (1a2e056) to head (373ce19).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #3572 +/- ## ========================================== - Coverage 62.54% 62.35% -0.19% ========================================== Files 898 899 +1 Lines 69010 69269 +259 Branches 3674 3688 +14 ========================================== + Hits 43160 43191 +31 - Misses 25162 25389 +227 - Partials 688 689 +1 ``` | [Flag](https://app.codecov.io/gh/thirdweb-dev/js/pull/3572/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) | Coverage Δ | | *Carryforward flag | |---|---|---|---| | [legacy_packages](https://app.codecov.io/gh/thirdweb-dev/js/pull/3572/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) | `65.59% <ø> (ø)` | | Carriedforward from [1a2e056](https://app.codecov.io/gh/thirdweb-dev/js/commit/1a2e0563299e22dbcac77d177a7b86011ecf377d?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) | | [packages](https://app.codecov.io/gh/thirdweb-dev/js/pull/3572/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) | `61.74% <0.00%> (-0.23%)` | :arrow_down: | | *This pull request uses carry forward flags. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) to find out more. | [Files](https://app.codecov.io/gh/thirdweb-dev/js/pull/3572?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) | Coverage Δ | | |---|---|---| | [.../web/wallets/shared/ConnectWalletSocialOptions.tsx](https://app.codecov.io/gh/thirdweb-dev/js/pull/3572?src=pr&el=tree&filepath=packages%2Fthirdweb%2Fsrc%2Freact%2Fweb%2Fwallets%2Fshared%2FConnectWalletSocialOptions.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev#diff-cGFja2FnZXMvdGhpcmR3ZWIvc3JjL3JlYWN0L3dlYi93YWxsZXRzL3NoYXJlZC9Db25uZWN0V2FsbGV0U29jaWFsT3B0aW9ucy50c3g=) | `25.74% <0.00%> (ø)` | | ... and [5 files with indirect coverage changes](https://app.codecov.io/gh/thirdweb-dev/js/pull/3572/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev)