ci: remove node 25 from test matrix#776
Conversation
Per vitest-dev/vitest#8757, vitest is throwing errors in node v25 due to some issues with the new webstorage api. Once this issue is resolved upstream, `latest` should be re-enabled.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #776 +/- ##
=======================================
Coverage 61.46% 61.46%
=======================================
Files 35 35
Lines 2948 2948
Branches 473 473
=======================================
Hits 1812 1812
Misses 1136 1136
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@JoshuaKGoldberg I feel like I've seen this is happening in some of the repos we share too(?), where it thinks it should be waiting on these steps, even though they've run. Have you been seeing this recently too (seems like just in the last day or two)
|
|
I think this happens when I run |
😕 These are not in the ruleset settings for |
|
Anything I need to do to move this forward? |
|
I poked around again and couldn't figure out the pending checks. Given that this is blocking #772 I don't think we should wait. Merged. Thanks as always @michaelfaith, you're a lifesaver! |

PR Checklist
main#768status: accepting prsOverview
Per vitest-dev/vitest#8757, vitest is throwing errors in node v25 due to some issues with the new webstorage api. Once this issue is resolved upstream,
latestshould be re-enabled.