Open agilgur5 opened 1 year ago
Left this as a Draft PR for now as it is breaking, and I think we should get in a patch release and stabilize MTS/CTS support beforehand.
And potentially there is more tech debt we want to remove in this PR as well; thought I might be missing something?
ooo tests caught that devDeps need updating too! awesome to see integration tests reaping rewards 🙂 updating Rollup to 3+ requires updating several plugins as well, so I'm gonna need to review some breaking changes for those
Proactively bumped the repro template to use Rollup v3 (and TS v5 and rpt2 v0.35)
Last time I tried to just update everything updatable, I got a lot of grief from tsjest
Yea I noticed from #447 😅 ESM adoption has (expectedly) caused a bit of churn in the ecosystem.
Though this PR specifically only needs to update all the Rollup deps, so maybe won't need updating the rest. But seems I might need to do so for #460 anyway
Summary
Bumping the Rollup peerDep and removing legacy tech debt workarounds
Details
Rollup upgrade
3.7.5
includes https://github.com/rollup/rollup/pull/4749buildEnd
workaround from #3732.60.0
includesthis.load
, so can remove thesatisfies
check2.14.0
includesthis.meta.watchMode
, so can remove the env checkOther tech debt removals
Figured I'd remove some other deprecated code / tech debt while I'm at it
rollupCommonJSResolveHack
objectHashIgnoreUnknownHack
warningobject-hash