-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Upgrade to typescript v5 #4184
Upgrade to typescript v5 #4184
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
ec806df
to
999c5ea
Compare
79b6d0f
to
79f3941
Compare
I think the workflow is overly restrictive. I believe that what we want is to check is that Hardhat's generated |
@alcuadrado that makes sense, but since it's passing, do you mind if we merge it? I think testing this properly would require e2e tests. I can update the e2e issue to include that, and we can remove that workflow when we do it. |
sure! |
We'll need this to be able to use viem.
@alcuadrado important things here:
~5.0.0
as the version range in all packages (even the toolbox).