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

build: sync cares source change #47359

Merged
merged 1 commit into from Apr 6, 2023
Merged

Conversation

gengjiawen
Copy link
Member

removed in #46415

@nodejs-github-bot
Copy link
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 Pull requests that update a dependency file. 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
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@gengjiawen gengjiawen added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 3, 2023
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@gengjiawen gengjiawen added the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 4, 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 4, 2023
@nodejs-github-bot
Copy link
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 An error occurred while landing this pull request using GitHub Actions. label Apr 4, 2023
@nodejs-github-bot
Copy link
Collaborator

@gengjiawen gengjiawen added the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 5, 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 5, 2023
@nodejs-github-bot
Copy link
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
Collaborator

@gengjiawen gengjiawen added commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Apr 6, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 6, 2023
@nodejs-github-bot nodejs-github-bot merged commit 0e79635 into nodejs:main Apr 6, 2023
53 checks passed
@nodejs-github-bot
Copy link
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>
@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 29, 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. cares Issues and PRs related to the c-ares dependency or the cares_wrap binding. commit-queue-failed An error occurred while landing this pull request using GitHub Actions. dependencies Pull requests that update a dependency file. 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants