Skip to content

test: change forEach to for - #66543

Closed
avivkeller wants to merge 1 commit into
nodejs:mainfrom
avivkeller:my-awesome-change
Closed

avivkeller wants to merge 1 commit into
nodejs:mainfrom
avivkeller:my-awesome-change

Conversation

@avivkeller

Copy link
Copy Markdown
Member

No description provided.

@avivkeller avivkeller added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Oct 5, 2026
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure. labels Oct 5, 2026
@codecov

codecov Bot commented Oct 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.40%. Comparing base (9746ebc) to head (5353302).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #66543   +/-   ##
=======================================
  Coverage   90.40%   90.40%           
=======================================
  Files         791      791           
  Lines      276120   276120           
  Branches    53022    53020    -2     
=======================================
+ Hits       249618   249623    +5     
- Misses      16897    16900    +3     
+ Partials     9605     9597    -8     

see 24 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.

@avivkeller avivkeller closed this Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants