Skip to content

Conversation

@wschurman
Copy link
Member

@wschurman wschurman commented Dec 1, 2025

Why

Upgrading the development version of this to 24 to match what the Expo server is currently using.

How

Upgrade versions.

Pin collect-v8-coverage to fix jestjs/jest#14766 until jest pulls in updated dependency.

Test Plan

CI

Copy link
Member Author

wschurman commented Dec 1, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@codecov
Copy link

codecov bot commented Dec 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (c453726) to head (381b9bc).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #323   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           87        87           
  Lines        12187     12187           
  Branches      1057      1057           
=========================================
  Hits         12187     12187           
Flag Coverage Δ
integration 7.52% <ø> (ø)
unittest 96.96% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@wschurman wschurman changed the base branch from wschurman/11-26-chore_upgrade_lru-cache to graphite-base/323 December 9, 2025 01:40
@wschurman wschurman force-pushed the wschurman/12-01-chore_upgrade_to_node_24_lts branch from 069cb0a to 01d04d0 Compare December 9, 2025 01:40
@graphite-app graphite-app bot changed the base branch from graphite-base/323 to main December 9, 2025 01:41
@wschurman wschurman force-pushed the wschurman/12-01-chore_upgrade_to_node_24_lts branch 2 times, most recently from 7f8fbe5 to 124bc18 Compare December 9, 2025 01:51
@wschurman wschurman force-pushed the wschurman/12-01-chore_upgrade_to_node_24_lts branch from 124bc18 to 381b9bc Compare December 9, 2025 02:32
@wschurman wschurman marked this pull request as ready for review December 9, 2025 02:38
@wschurman wschurman merged commit 767d951 into main Dec 9, 2025
4 checks passed
@wschurman wschurman deleted the wschurman/12-01-chore_upgrade_to_node_24_lts branch December 9, 2025 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: v8 coverage lowers when more tests are executed

4 participants