fix/deps: upgrade rollup-plugin-typescript2 to fix cache issue#896
Conversation
This comment has been minimized.
This comment has been minimized.
|
Thanks for the patch @tanem -- was actually looking to bump this myself once rpts2 v0.28.0 is released, because that will contain my fix that was recently merged: ezolenko/rollup-plugin-typescript2#221 (which resolves a few issues and does a root cause fix for a PR of mine in TSDX). Though that requires a few extra changes and some testing, so maybe I'll do that as a separate PR then. But was planning to fix both / upgrade all the way up for TSDX v0.14.1.
If that didn't work for you could you provide more details on that? Neither nuking the cache nor all of
💯
I've temporarily disabled it because all the dep updates were spammy (duplicate PRs, unnecessarily updating patches/minors, updating peerDeps that can't be updated in isolation, etc). Need to add a bunch of filters to it, but even then, it's not feasible, not always necessary, and may actually be a bad idea stability-wise to update deps immediately after release. |
|
Hey @agilgur5 👋 Here's what I did to get the issue: Then I tried: Finally: Once I applied the update in this PR to my local copy of On the topic of Dependabot, have you considered using Renovate instead? I find it gives a lot more control in terms of how much noise you have to deal with as a maintainer. It also has a One thing it doesn't do that Dependabot does do is update transient dependencies for security updates (hopefully I described that correctly, don't want to misrepresent Renovate 😅). In those cases, I've seen other open-source repos use Dependabot just for security patches, and Renovate for the rest. For my own repos I usually manually update if I need a security patch applied promptly, otherwise I let the usual Renovate update cycle handle it. |
|
Thanks for the additional details @tanem
This is pretty intriguing; I thought most folks, like Jared, were getting this during an upgrade of TSDX, but this is happening to you on a fresh install.... yea that's even more confusing since rpts2 hasn't been updated since TSDX v0.13.0 / #506 . And #691 / #758 were the only changes to the options and were in v0.13.3... but for some reason this wasn't discovered until v0.14.0 🤔 🤔 Will have to follow up on this upstream |
I have not. I didn't realize Renovate did dependencies, I thought it was just an automated merge bot based on past usage I'd seen. Also recently switched from Greenkeeper to have even more issues with Snyk (as linked above), then to Dependabot (#846 ), so I'm not clamoring to change and learn another potentially problematic bot, especially if I need to use two for security issues (which are the few that are quite important / aren't noise) 😕 But I'll keep it in my mind for the future, thanks |
|
Reproduced and confirmed #892 as well as did some more investigations there, including finding that this bug did start with TSDX v0.13.3, just with less popular combinations of formats than v0.14.0 for unknown reason. I'm not going to wait on the release of ezolenko/rollup-plugin-typescript2#221 / rtps2 v0.28.0 anymore since this issue seems to be quite widespread, #898 and #900 hit into it as well. I'll probably merge and release TSDX v0.14.1 tomorrow, but I'd like to add a smoke test that tests a build of all formats simultaneously to confirm this works and doesn't regress. Apparently there aren't any tests of |
- fixes a cache issue with certain builds by upgrading to the most
recent version of rollup-plugin-typescript2, v0.27.3, which just had
a PR and release for this specific cache issue 2 weeks ago
- there were no relevant breaking changes in the upgrade from rpts2
v0.26 to v0.27.0, so this should be a straight bugfix
- there was never a test that built all, meaning tests left out the
somewhat popular UMD build in particular
- per my investigation, CJS + UMD, CJS + System, UMD + System started
bugging out recently due to an upstream bug, but this wasn't known
proactively because there were no tests for it
- and I also found that CJS + System actually was bugging out in
the previous version, TSDX v0.13.3, but I'm guessing no one
reported it back then as it's an unpopular combination of formats
- this test fails prior to upgrade of rpts2 to v0.27.3 and succeeds
after the upgrade
- so this should act as a regression test against this bug as well
- created a new e2e build-options test file for this because
build-default is meant to have 0 options and test only the defaults
- had to give it a differentiated "stage" name as it uses the same
build-default fixture and so breaks during test parallelization
without that
- should also move the regeneratorRuntime `--target node` test here
since that's an option and not a default
3845d36 to
8f32645
Compare
agilgur5
left a comment
There was a problem hiding this comment.
New test passes, no weird logs in the output (aside from #884 (comment) which is an existing issue). Test conforms to existing test style and pretty straightforward so that LGTM to me as well.
|
@allcontributors please add @tanem for code, bug (the |
|
I've put up a pull request to add @tanem! 🎉 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Thanks for the all the work here and the merge @agilgur5 🙏 |
Bumped into #888/#892 when creating a new repo, and the workaround mentioned here didn't work for me.
This patch brings in the permanent fix from ezolenko/rollup-plugin-typescript2#243. Tested by yarn linking tsdx across repos, and the build looks to be working ok with this change.
Guess dependabot would have eventually updated this dep, but figured it was important enough to do ASAP?