Add test-install CI job to test packaging, installation and importing of rollup package#6171
Add test-install CI job to test packaging, installation and importing of rollup package#6171lukastaegert merged 4 commits intorollup:masterfrom antoninkriz:test-install
Conversation
|
@antoninkriz is attempting to deploy a commit to the rollup-js Team on Vercel. A member of the Team first needs to authorize it. |
lukastaegert
left a comment
There was a problem hiding this comment.
Thanks, that is actually a nice idea! Will think a little about this before merging as it is late today already at my place. Hope the fix goes through well.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #6171 +/- ##
=======================================
Coverage 98.74% 98.74%
=======================================
Files 271 271
Lines 10639 10639
Branches 2847 2847
=======================================
Hits 10505 10505
Misses 91 91
Partials 43 43 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
5369863 to
96b5453
Compare
lukastaegert
left a comment
There was a problem hiding this comment.
Thanks again and sorry for the wait. I took the liberty to inline the install test into the regular test runs as that saves us some runners and there is a lot of overhead just to get to that point. If that works, I will merge it.
|
This PR has been released as part of rollup@4.53.3. You can test it via |
This PR contains:
Are tests included?
Breaking Changes?
List any relevant issue numbers:
Description
This PR adds CI job to test packaging, installation and import of rollup package. This check should warn against future occurrences of issues similar to #6168.
Very hacky pipeline runs which shows:
4.53.0https://github.com/antoninkriz/rollup/actions/runs/19178593890/job/54829691567
4.52.5https://github.com/antoninkriz/rollup/actions/runs/19179731676/job/54834016969