-
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
chore: integrate vitest matchers globally #3425
chore: integrate vitest matchers globally #3425
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@starc007 is attempting to deploy a commit to the Fuel Labs Team on Vercel. A member of the Team first needs to authorize it. |
CodSpeed Performance ReportMerging #3425 will improve performances by 47.77%Comparing Summary
Benchmarks breakdown
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @starc007,
Thank you so much for this contribution. We really appreciate it 🙏
It appears that the current solution hasn’t resolved the TS errors.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR also needs a changeset, please run pnpm changeset
from the root directory.
5b0f72a
to
b70acc1
Compare
Hey @Torres-ssf Issue: This suggests the Questions:
Any guidance on the correct way to set this up would be appreciated. |
pushed those changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one @starc007 🚀
Old review - now fixed :)
vitest
matchers globally #3421Summary
Checklist