Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: V8: cherry-pick 8e10685ff918 #47440

Merged
merged 1 commit into from
Apr 11, 2023
Merged

Conversation

gengjiawen
Copy link
Member

Original commit message:

[build]: fix wrong name

Change-Id: I504e836da1e36d7b9e504dbdbff8803363f08ccd
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4392064
Reviewed-by: Tobias Tebbi <tebbi@chromium.org>
Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
Cr-Commit-Position: refs/heads/main@{#86919}

Refs: v8/v8@8e10685

Original commit message:

    [build]: fix wrong name

    Change-Id: I504e836da1e36d7b9e504dbdbff8803363f08ccd
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4392064
    Reviewed-by: Tobias Tebbi <tebbi@chromium.org>
    Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#86919}

Refs: v8/v8@8e10685
@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/v8-update

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency. labels Apr 6, 2023
@gengjiawen gengjiawen added request-ci Add this label to start a Jenkins CI on a PR. and removed build Issues and PRs related to build files or the CI. v8 engine Issues and PRs related to the V8 dependency. needs-ci PRs that need a full CI run. labels Apr 6, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 6, 2023
@nodejs-github-bot
Copy link
Collaborator

@gengjiawen gengjiawen added v8 engine Issues and PRs related to the V8 dependency. build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Apr 6, 2023
@nodejs-github-bot
Copy link
Collaborator

@targos targos added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 8, 2023
@gengjiawen gengjiawen requested review from richardlau and Trott April 11, 2023 03:29
@gengjiawen gengjiawen added the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 11, 2023
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Apr 11, 2023
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/47440
✔  Done loading data for nodejs/node/pull/47440
----------------------------------- PR info ------------------------------------
Title      deps: V8: cherry-pick 8e10685ff918 (#47440)
Author     Jiawen Geng  (@gengjiawen)
Branch     gengjiawen:v8/pick -> nodejs:main
Labels     build, v8 engine, author ready, needs-ci
Commits    1
 - deps: V8: cherry-pick 8e10685ff918
Committers 1
 - Jiawen Geng 
PR-URL: https://github.com/nodejs/node/pull/47440
Refs: https://github.com/v8/v8/commit/8e10685ff9187ab1b4041a1b320dc6012c09c460
Reviewed-By: Michaël Zasso 
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/47440
Refs: https://github.com/v8/v8/commit/8e10685ff9187ab1b4041a1b320dc6012c09c460
Reviewed-By: Michaël Zasso 
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 06 Apr 2023 02:09:00 GMT
   ✔  Approvals: 3
   ✔  - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/47440#pullrequestreview-1376735937
   ✔  - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/47440#pullrequestreview-1379058117
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/47440#pullrequestreview-1379131272
   ✘  Last GitHub CI failed
   ℹ  Last Full PR CI on 2023-04-11T14:32:53Z: https://ci.nodejs.org/job/node-test-pull-request/51133/
- Querying data for job/node-test-pull-request/51133/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/4670174219

@nodejs-github-bot
Copy link
Collaborator

@debadree25 debadree25 added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Apr 11, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 11, 2023
@nodejs-github-bot nodejs-github-bot merged commit 4bcb42c into nodejs:main Apr 11, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in 4bcb42c

targos pushed a commit that referenced this pull request May 2, 2023
Original commit message:

    [build]: fix wrong name

    Change-Id: I504e836da1e36d7b9e504dbdbff8803363f08ccd
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4392064
    Reviewed-by: Tobias Tebbi <tebbi@chromium.org>
    Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#86919}

Refs: v8/v8@8e10685
PR-URL: #47440
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@targos targos mentioned this pull request May 2, 2023
@danielleadams danielleadams added the dont-land-on-v18.x PRs that should not land on the v18.x-staging branch and should not be released in v18.x. label Jun 27, 2023
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. build Issues and PRs related to build files or the CI. dont-land-on-v18.x PRs that should not land on the v18.x-staging branch and should not be released in v18.x. needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants