-
-
Notifications
You must be signed in to change notification settings - Fork 540
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: tidy up dependencies #2480
Conversation
✅ Deploy Preview for biomejs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Parser conformance results onjs/262
jsx/babel
symbols/microsoft
ts/babel
ts/microsoft
|
CodSpeed Performance ReportMerging #2480 will improve performances by 6.67%Comparing Summary
Benchmarks breakdown
|
I'm having some weird issues when running At first it emits:
And now it emits sth like:
And sometimes it can be built successfully. Is this related to this PR? This issue now breaks the website Edit: Ah, nevermind. I think this is caused by the cached target dir. It works when I delete it and rebuild everything. |
Summary
This PR removes a few duplicate crates around the codebase.
Also, it upgrades
hashbrown
to the latest, which has a breaking change. The new methodextract_if
doesn'tDrop
the elements anymore, so now thedrop
is implemented manually.Test Plan
The manual
drop
doesn't break a test that was failing on my machine.