Skip to content

src: merge CreateEnvironment() overloads#63596

Open
addaleax wants to merge 1 commit into
nodejs:mainfrom
addaleax:abi-break-CreateEnvironment
Open

src: merge CreateEnvironment() overloads#63596
addaleax wants to merge 1 commit into
nodejs:mainfrom
addaleax:abi-break-CreateEnvironment

Conversation

@addaleax
Copy link
Copy Markdown
Member

API-compatible, but ABI-breaking, so done separately from the original 3090def.

@addaleax addaleax added c++ Issues and PRs that require attention from people who are familiar with C++. semver-major PRs that contain breaking changes and should be released in the next major version. labels May 27, 2026
@nodejs-github-bot nodejs-github-bot added the needs-ci PRs that need a full CI run. label May 27, 2026
@addaleax addaleax force-pushed the abi-break-CreateEnvironment branch from 23c5506 to 37016f3 Compare May 27, 2026 09:04
@addaleax addaleax changed the title src: merge CreateEnvironment() implementations src: merge CreateEnvironment() overloads May 27, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.32%. Comparing base (8257091) to head (a21167c).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #63596   +/-   ##
=======================================
  Coverage   90.32%   90.32%           
=======================================
  Files         730      730           
  Lines      234671   234662    -9     
  Branches    43946    43958   +12     
=======================================
+ Hits       211965   211967    +2     
  Misses      14423    14423           
+ Partials     8283     8272   -11     
Files with missing lines Coverage Δ
src/api/environment.cc 78.72% <ø> (-0.30%) ⬇️
src/node.h 92.30% <ø> (ø)

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

Copy link
Copy Markdown
Member

@mertcanaltin mertcanaltin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@legendecas
Copy link
Copy Markdown
Member

The linter error should be fixed with #63598 landed on the main branch. Would you mind rebasing on top of the main branch?

API-compatible, but ABI-breaking, so done separately from the
original 3090def.

Signed-off-by: Anna Henningsen <anna@addaleax.net>
@addaleax
Copy link
Copy Markdown
Member Author

@legendecas Thank you, done!

@addaleax addaleax force-pushed the abi-break-CreateEnvironment branch from 37016f3 to a21167c Compare May 27, 2026 19:20
@legendecas legendecas added the request-ci Add this label to start a Jenkins CI on a PR. label May 27, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 27, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

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

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. semver-major PRs that contain breaking changes and should be released in the next major version.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants