Skip to content

deps: update undici to 5.25.2 - #49559

Merged
nodejs-github-bot merged 3 commits into
mainfrom
actions/tools-update-undici
Sep 23, 2023
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

@nodejs-github-bot nodejs-github-bot commented Sep 8, 2023

Copy link
Copy Markdown
Collaborator

This is an automated update of undici to 5.25.2.

@nodejs-github-bot nodejs-github-bot added dependencies PRs that add, update, or configure Node.js dependencies. lib / src Issues and PRs involving general changes in the lib/ or src/ directories. needs-ci PRs that need a full CI run. labels Sep 8, 2023
@mcollina mcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 8, 2023
@mcollina
mcollina requested review from KhafraDev and ronag September 8, 2023 15:20
@github-actions 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

Copy link
Copy Markdown
Collaborator Author

@aduh95

aduh95 commented Sep 8, 2023

Copy link
Copy Markdown
Contributor

Quite a few seemingly related test failures

@KhafraDev

Copy link
Copy Markdown
Member

is it only builds without ssl?

@nodejs-github-bot nodejs-github-bot changed the title deps: update undici to 5.24.0 deps: update undici to 5.25.0 Sep 20, 2023
@mcollina mcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 21, 2023

@mcollina 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.

lgtm

@github-actions 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

Copy link
Copy Markdown
Collaborator Author

@nodejs-github-bot nodejs-github-bot changed the title deps: update undici to 5.25.0 deps: update undici to 5.25.2 Sep 22, 2023
@mcollina

Copy link
Copy Markdown
Member

@KhafraDev let's see if WebSocket runs :)

@KhafraDev

Copy link
Copy Markdown
Member

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

@mcollina mcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 23, 2023
@github-actions 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

Copy link
Copy Markdown
Collaborator Author

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator Author

@mcollina mcollina added commit-queue-squash PRs the Commit Queue should land as one squashed commit. commit-queue PRs queued for automated landing through the Commit Queue. and removed needs-ci PRs that need a full CI run. labels Sep 23, 2023

@mcollina 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.

lgtm

@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Sep 23, 2023
@nodejs-github-bot
nodejs-github-bot merged commit ef062b9 into main Sep 23, 2023
@nodejs-github-bot
nodejs-github-bot deleted the actions/tools-update-undici branch September 23, 2023 11:40
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator Author

Landed in ef062b9

ruyadorno pushed a commit that referenced this pull request Sep 28, 2023
PR-URL: #49559
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Matthew Aitken <maitken033380023@gmail.com>
This was referenced Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue-squash PRs the Commit Queue should land as one squashed commit. dependencies PRs that add, update, or configure Node.js dependencies. lib / src Issues and PRs involving general changes in the lib/ or src/ directories.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unexpected error when using signal with reason in fetch

5 participants