Skip to content

Remove calls to APIs deprecated in V8 12.6 #52991

Closed
Closed
Remove calls to APIs deprecated in V8 12.6#52991
Labels
deprecationsIssues and PRs related to deprecations.v8 engineIssues and PRs related to the V8 dependency.

Description

@ajklein
ajklein
opened on May 14, 2024

Both updates are trivial:

  • v8::ScriptOrigin's constructor no longer takes an Isolate (it was already unused)
  • v8::SafeForTerminationScope and the Isolate creation param only_terminate_in_safe_scope are gone (they were already no-ops)

See v8#185 for the fixes I made to keep V8's CI happy.

cc @joyeecheung

Activity

  1. marco-ippolito
    added
    v8 engineIssues and PRs related to the V8 dependency.
    deprecationsIssues and PRs related to deprecations.
    on May 15, 2024
  2. targos
    added a commit that references this issue on May 15, 2024

    src: remove calls to recently deprecated V8 APIs

    b584849

  3. targos commented on May 15, 2024

    @targos
    targos
    on May 15, 2024
    Member

    Thanks for the heads up! I opened #52996 with your changes.

  4. nodejs-github-bot
    closed this as completedin a6d54f1on May 19, 2024
  5. pull
    added a commit that references this issue on May 19, 2024

    src: remove calls to recently deprecated V8 APIs

    a6d54f1
  6. targos
    added a commit that references this issue on May 21, 2024

    src: remove calls to recently deprecated V8 APIs

    d750a3c
  7. soophoo
    added a commit that references this issue on Jun 20, 2024

    src: remove calls to recently deprecated V8 APIs

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    deprecationsIssues and PRs related to deprecations.v8 engineIssues and PRs related to the V8 dependency.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions