Skip to content

build: sync cares source change - #47359

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
gengjiawen:fix_cares
Apr 6, 2023
Merged

build: sync cares source change#47359
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
gengjiawen:fix_cares

Conversation

@gengjiawen

Copy link
Copy Markdown
Member

removed in #46415

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/net

@nodejs-github-bot nodejs-github-bot added cares Issues and PRs related to the c-ares dependency or the cares_wrap binding. dependencies PRs that add, update, or configure Node.js dependencies. needs-ci PRs that need a full CI run. labels Apr 1, 2023
@gengjiawen gengjiawen added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 1, 2023
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 1, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@gengjiawen gengjiawen added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Apr 3, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@gengjiawen gengjiawen added the commit-queue PRs queued for automated landing through the Commit Queue. label Apr 4, 2023
@nodejs-github-bot nodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Apr 4, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/47359
✔  Done loading data for nodejs/node/pull/47359
----------------------------------- PR info ------------------------------------
Title      build: sync cares source change (#47359)
Author     Jiawen Geng  (@gengjiawen)
Branch     gengjiawen:fix_cares -> nodejs:main
Labels     cares, author ready, needs-ci, dependencies
Commits    1
 - build: sync cares source change
Committers 1
 - gengjiawen 
PR-URL: https://github.com/nodejs/node/pull/47359
Reviewed-By: Richard Lau 
Reviewed-By: Luigi Pinca 
Reviewed-By: Rich Trott 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/47359
Reviewed-By: Richard Lau 
Reviewed-By: Luigi Pinca 
Reviewed-By: Rich Trott 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sat, 01 Apr 2023 08:03:09 GMT
   ✔  Approvals: 3
   ✔  - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/47359#pullrequestreview-1367841805
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/47359#pullrequestreview-1368127356
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/47359#pullrequestreview-1369507051
   ✘  Last GitHub CI failed
   ℹ  Last Full PR CI on 2023-04-04T01:56:09Z: https://ci.nodejs.org/job/node-test-pull-request/50890/
- Querying data for job/node-test-pull-request/50890/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/4604897824

@gengjiawen gengjiawen removed the commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. label Apr 4, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@gengjiawen gengjiawen added the commit-queue PRs queued for automated landing through the Commit Queue. label Apr 5, 2023
@nodejs-github-bot nodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Apr 5, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/47359
✔  Done loading data for nodejs/node/pull/47359
----------------------------------- PR info ------------------------------------
Title      build: sync cares source change (#47359)
Author     Jiawen Geng  (@gengjiawen)
Branch     gengjiawen:fix_cares -> nodejs:main
Labels     cares, author ready, needs-ci, dependencies
Commits    1
 - build: sync cares source change
Committers 1
 - gengjiawen 
PR-URL: https://github.com/nodejs/node/pull/47359
Reviewed-By: Richard Lau 
Reviewed-By: Luigi Pinca 
Reviewed-By: Rich Trott 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/47359
Reviewed-By: Richard Lau 
Reviewed-By: Luigi Pinca 
Reviewed-By: Rich Trott 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sat, 01 Apr 2023 08:03:09 GMT
   ✔  Approvals: 3
   ✔  - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/47359#pullrequestreview-1367841805
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/47359#pullrequestreview-1368127356
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/47359#pullrequestreview-1369507051
   ✘  Last GitHub CI failed
   ℹ  Last Full PR CI on 2023-04-04T07:04:31Z: https://ci.nodejs.org/job/node-test-pull-request/50905/
- Querying data for job/node-test-pull-request/50905/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/4618985299

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@gengjiawen gengjiawen added commit-queue PRs queued for automated landing through the Commit Queue. commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Apr 6, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Apr 6, 2023
@nodejs-github-bot
nodejs-github-bot merged commit 0e79635 into nodejs:main Apr 6, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 0e79635

@gengjiawen
gengjiawen deleted the fix_cares branch April 6, 2023 01:15
RafaelGSS pushed a commit that referenced this pull request Apr 13, 2023
PR-URL: #47359
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. cares Issues and PRs related to the c-ares dependency or the cares_wrap binding. commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. dependencies PRs that add, update, or configure Node.js dependencies. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants