Skip to content

Conversation

0-wiz-0
Copy link
Contributor

@0-wiz-0 0-wiz-0 commented Sep 2, 2025

Include missing cmath.h header.

Fixes: #59714

Include missing cmath.h header.

Fixes: nodejs#59714
@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. labels Sep 2, 2025
Copy link

codecov bot commented Sep 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.92%. Comparing base (961554c) to head (510e366).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #59718      +/-   ##
==========================================
- Coverage   89.93%   89.92%   -0.02%     
==========================================
  Files         667      667              
  Lines      196775   196775              
  Branches    38409    38413       +4     
==========================================
- Hits       176977   176951      -26     
+ Misses      12247    12241       -6     
- Partials     7551     7583      +32     
Files with missing lines Coverage Δ
src/tracing/traced_value.cc 94.91% <ø> (ø)

... and 34 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.

@targos targos added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 2, 2025
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 2, 2025
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@addaleax addaleax added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 2, 2025
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@richardlau richardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 3, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 4, 2025
@nodejs-github-bot nodejs-github-bot merged commit 5600c0e into nodejs:main Sep 4, 2025
73 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 5600c0e

targos pushed a commit that referenced this pull request Sep 4, 2025
Include missing cmath.h header.

Fixes: #59714
PR-URL: #59718
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Ulises Gascón <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
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. c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fpclassify namespace specification breaks build on NetBSD
7 participants