Skip to content

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
@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
@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 Trott and richardlau 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 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
@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
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. needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency.

7 participants