src: fix usage of deprecated V8 API by targos · Pull Request #60174 · nodejs/node · GitHub
Skip to content

src: fix usage of deprecated V8 API#60174

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
targos:embedderdatav2
Oct 11, 2025
Merged

src: fix usage of deprecated V8 API#60174
nodejs-github-bot merged 1 commit intonodejs:mainfrom
targos:embedderdatav2

Conversation

@targos
Copy link
Copy Markdown
Member

@targos targos commented Oct 9, 2025

Namely GetContinuationPreservedEmbedderData and
SetContinuationPreservedEmbedderData for their "V2" variants.

Refs: v8/v8@da41db3

Namely `GetContinuationPreservedEmbedderData` and
`SetContinuationPreservedEmbedderData` for their "V2" variants.

Refs: v8/v8@da41db3
@targos targos added dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v24.x-staging branch and should not be released in v24.x. labels Oct 9, 2025
@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 Oct 9, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 9, 2025

@addaleax addaleax added the request-ci Add this label to start a Jenkins CI on a PR. label Oct 9, 2025
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 9, 2025
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@targos
Copy link
Copy Markdown
Member Author

targos commented Oct 10, 2025

@richardlau
Copy link
Copy Markdown
Member

macOS failed on https://ci.nodejs.org/job/node-test-commit-osx/67253/nodes=macos15-x64/testReport/junit/(root)/parallel/test_without_async_context_frame/, so I'm not very confident with this now.

FWIW that test has been flaky across 19 PRs: https://github.com/nodejs/reliability/blob/main/reports/2025-10-10.md#jstest-failure

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@targos targos added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 10, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 11, 2025
@nodejs-github-bot nodejs-github-bot merged commit 367bcce into nodejs:main Oct 11, 2025
81 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

c++ Issues and PRs that require attention from people who are familiar with C++. dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v24.x-staging branch and should not be released in v24.x. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants