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

feat!: upgrade llhttp to 9.2.0 (#2705) #3388

Merged
merged 3 commits into from
Jul 5, 2024
Merged

feat!: upgrade llhttp to 9.2.0 (#2705) #3388

merged 3 commits into from
Jul 5, 2024

Conversation

metcoder95
Copy link
Member

  • upgrade llhttp

  • fix tests

  • set version of llhttp 9.2.0

This relates to...

Rationale

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

* upgrade llhttp

* fix tests

* set version of llhttp 9.2.0
@ronag ronag requested a review from mcollina July 3, 2024 09:28
@metcoder95 metcoder95 changed the title chore: upgrade llhttp to 9.2.0 (#2705) feat!: upgrade llhttp to 9.2.0 (#2705) Jul 3, 2024
@metcoder95 metcoder95 marked this pull request as ready for review July 3, 2024 10:21
@metcoder95 metcoder95 mentioned this pull request Jul 3, 2024
6 tasks
@Uzlopak
Copy link
Contributor

Uzlopak commented Jul 3, 2024

Doesnt next already contain 9.2.0

@mcollina
Copy link
Member

mcollina commented Jul 3, 2024

We should really just merge next into main.

@ronag
Copy link
Member

ronag commented Jul 3, 2024

We should really just merge next into main.

We tried. The merge looks super weird. Next branch only contains 2 PRs we need.

@metcoder95
Copy link
Member Author

We should really just merge next into main.

We tried. The merge looks super weird. Next branch only contains 2 PRs we need.

It contains too many conflicts that might be hard to solve, overall only the two PRs listed here are the ones we need to land on main.

@ronag ronag merged commit be0703b into main Jul 5, 2024
36 checks passed
@Uzlopak Uzlopak deleted the feat/llhttp branch July 5, 2024 10:01
@github-actions github-actions bot mentioned this pull request Dec 3, 2024
This was referenced Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants