Skip to content

Conversation

@legendecas
Copy link
Member

@legendecas legendecas commented Nov 25, 2025

Tests should run with the node exe built in the specified BUILDTYPE.

This depends on #60806.

Refs: #60806 (comment)

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Nov 25, 2025
@legendecas legendecas marked this pull request as draft November 25, 2025 12:05
@legendecas legendecas marked this pull request as ready for review November 25, 2025 12:07
@codecov
Copy link

codecov bot commented Nov 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.04%. Comparing base (768f3ba) to head (936cdea).
⚠️ Report is 13 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #60850      +/-   ##
==========================================
- Coverage   88.56%   88.04%   -0.53%     
==========================================
  Files         703      703              
  Lines      208291   208291              
  Branches    40169    40087      -82     
==========================================
- Hits       184472   183384    -1088     
- Misses      15831    16847    +1016     
- Partials     7988     8060      +72     

see 95 files with indirect coverage changes

🚀 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.

@legendecas legendecas force-pushed the embed-debug branch 2 times, most recently from e7de0f6 to c03895a Compare November 25, 2025 13:48
@jakecastelli jakecastelli added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Nov 28, 2025
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@legendecas
Copy link
Member Author

@cjihrig @vmoroz I rebased this PR after #60806 landed and the CI is green now. Would you mind taking a look again? Thank you!

@legendecas legendecas added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 1, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 1, 2025
@nodejs-github-bot nodejs-github-bot merged commit 28142a6 into nodejs:main Dec 1, 2025
61 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 28142a6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants