src: merge `CreateEnvironment()` overloads by addaleax · Pull Request #63596 · nodejs/node · GitHub
Skip to content

src: merge CreateEnvironment() overloads#63596

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
addaleax:abi-break-CreateEnvironment
Jun 4, 2026
Merged

src: merge CreateEnvironment() overloads#63596
nodejs-github-bot merged 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

codecov Bot commented May 27, 2026

Copy link
Copy Markdown

@mertcanaltin mertcanaltin left a comment

Copy link
Copy Markdown
Member

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

nodejs-github-bot commented May 28, 2026

Copy link
Copy Markdown
Collaborator

@addaleax

addaleax commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

@nodejs/tsc This is technically semver-major because of the ABI change so needs another TSC approval

@gurgunday gurgunday left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@gurgunday gurgunday added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 3, 2026
@addaleax addaleax added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 4, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 4, 2026
@nodejs-github-bot nodejs-github-bot merged commit 2d0c926 into nodejs:main Jun 4, 2026
77 checks passed
@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

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

6 participants