-
Notifications
You must be signed in to change notification settings - Fork 110
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 client dependencies - June 2023 #2582
Upgrade client dependencies - June 2023 #2582
Conversation
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.
nit: Rogue line addition?
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.
Woops, thanks for catching!
@ethanlook I had to revert the prime upgrade as well because of |
Interesting, my understanding is that this only affects install from prime, not other projects. We definitely don't want to only allow consumers to use pnpm. Edit: 🤦🏻♂️ duh, preinstall runs when installing prime in other projects. I'll put up a PR to revert that line. |
@Otterverse Would you happen to know why so many checks are failing here? CI is giving issues unrelated to these changes. |
I think @edaniels broke it. :-) But also fixed it a short bit ago. faf57ab Rebase on main and try again. |
…ade-client-deps-jun-23
Code Coverage
|
This most notably upgrades us from Svelte 3 to 4. I had to hold off on upgrading THREE. There's an unknown issue from r152 -> r153 that disabled vertex colors on point clouds, causing probabilistic gray-scaling rendering of points to fail. I'll investigate this separately.