FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

deps: update undici to 5.25.2 by nodejs-github-bot · Pull Request #49559 · nodejs/node · GitHub

/ node Public

deps: update undici to 5.25.2 - #49559

Merged
nodejs-github-bot merged 3 commits into
mainfrom
actions/tools-update-undici
Sep 23, 2023
ConversationCommits (3)ChecksFiles changed
Merged

deps: update undici to 5.25.2#49559
nodejs-github-bot merged 3 commits into
mainfrom
actions/tools-update-undici

Conversation

nodejs-github-bot commented Sep 8, 2023 •
edited
Loading

Copy link
Copy Markdown
Collaborator

This is an automated update of undici to 5.25.2.

nodejs-github-bot added dependencies Pull requests that update a dependency file. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Sep 8, 2023
mcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 8, 2023
mcollina requested review from KhafraDev and ronag September 8, 2023 15:20
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 8, 2023

nodejs-github-bot commented Sep 8, 2023

Copy link
Copy Markdown
Collaborator Author

CI: https://ci.nodejs.org/job/node-test-pull-request/53819/

aduh95 commented Sep 8, 2023

Copy link
Copy Markdown
Contributor

Quite a few seemingly related test failures

KhafraDev commented Sep 8, 2023

Copy link
Copy Markdown
Member

is it only builds without ssl?

This was referenced Sep 9, 2023
Open
Open
Open
joyeecheung mentioned this pull request Sep 11, 2023
Closed
github-actions Bot mentioned this pull request Sep 12, 2023
Open
19 tasks
atlowChemi mentioned this pull request Sep 13, 2023
Closed
atlowChemi linked an issue Sep 13, 2023 that may be closed by this pull request
Unexpected error when using signal with reason in fetch #49557
Closed
KhafraDev mentioned this pull request Sep 19, 2023
Merged
nodejs-github-bot changed the title deps: update undici to 5.24.0 deps: update undici to 5.25.0 Sep 20, 2023
mcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 21, 2023
mcollina approved these changes Sep 21, 2023
View reviewed changes

mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

lgtm

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 21, 2023

nodejs-github-bot commented Sep 21, 2023

Copy link
Copy Markdown
Collaborator Author

CI: https://ci.nodejs.org/job/node-test-pull-request/54106/

github-actions Bot mentioned this pull request Sep 22, 2023
Open
17 tasks
nodejs-github-bot changed the title deps: update undici to 5.25.0 deps: update undici to 5.25.2 Sep 22, 2023

mcollina commented Sep 22, 2023

Copy link
Copy Markdown
Member

@KhafraDev let's see if WebSocket runs :)

KhafraDev commented Sep 22, 2023

Copy link
Copy Markdown
Member

I tested it locally, there is something very strange going on if it doesn't work lol

mcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 23, 2023
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 23, 2023

nodejs-github-bot commented Sep 23, 2023

Copy link
Copy Markdown
Collaborator Author

CI: https://ci.nodejs.org/job/node-test-pull-request/54170/

github-actions Bot mentioned this pull request Sep 23, 2023
Open
16 tasks

nodejs-github-bot commented Sep 23, 2023

Copy link
Copy Markdown
Collaborator Author

CI: https://ci.nodejs.org/job/node-test-pull-request/54175/

mcollina added commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. commit-queue Add this label to land a pull request using GitHub Actions. and removed needs-ci PRs that need a full CI run. labels Sep 23, 2023
mcollina approved these changes Sep 23, 2023
View reviewed changes

mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

lgtm

nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 23, 2023
nodejs-github-bot merged commit ef062b9 into main Sep 23, 2023
nodejs-github-bot deleted the actions/tools-update-undici branch September 23, 2023 11:40

nodejs-github-bot commented Sep 23, 2023

Copy link
Copy Markdown
Collaborator Author

Landed in ef062b9

github-actions Bot mentioned this pull request Sep 24, 2023
Open
20 tasks
joyeecheung mentioned this pull request Sep 25, 2023
Closed
ruyadorno pushed a commit that referenced this pull request Sep 28, 2023
deps: update undici to 5.24.0
61d18d6 
PR-URL: #49559
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Matthew Aitken <maitken033380023@gmail.com>
This was referenced Sep 28, 2023
Closed
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

mcollina mcollina approved these changes
KhafraDev KhafraDev approved these changes
ronag Awaiting requested review from ronag

Assignees

No one assigned

Labels

commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. dependencies Pull requests that update a dependency file. lib / src Issues and PRs related to general changes in the lib or src directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Unexpected error when using signal with reason in fetch

5 participants

Back | FazBrowse Home | New Git URL