Update Vite (partial [SYNTH-145])#1388
Conversation
7dc3519 to
f56a857
Compare
f56a857 to
eb1ce02
Compare
rutmanz
left a comment
There was a problem hiding this comment.
tests are failing, but also is there a reason to go one dependency at a time rather than all at once? A lot of these things have peer dependencies on different versions so to me it makes sense to try and upgrade everything in one PR
Part of [SYNTH-145][SYNTH-145])
I think that updating our testing code should be separate from updating our actual UI, so that there is less opportunity to miss regressions.
Sure it makes sense if the dependencies are peers of each other, then we should update them together, but that isn't the case for vite. it only depends on the vite plugins |
Task
part of SYNTH-145
Symptom
Solution
Verification
Before merging, ensure the following criteria are met: