Skip to content

llhttp: update to 1.1.2 - #27513

Closed
indutny wants to merge 1 commit into
nodejs:masterfrom
indutny:feature/update-llhttp-1.1.2
Closed

llhttp: update to 1.1.2#27513
indutny wants to merge 1 commit into
nodejs:masterfrom
indutny:feature/update-llhttp-1.1.2

Conversation

@indutny

Copy link
Copy Markdown
Member
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

cc @nodejs/http

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@indutny
indutny requested a review from cjihrigMay 1, 2019 09:22

@cjihrigcjihrig left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, but the commit message is failing linting. Maybe try deps: update llhttp to 1.1.2.

@indutny
indutnyforce-pushed the feature/update-llhttp-1.1.2 branch from 869f8e8 to 11a9d53CompareMay 1, 2019 18:07
@indutny

Copy link
Copy Markdown
MemberAuthor

@cjihrig 🤦‍♂ my bad. Thanks for pointing out!

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 2, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@targos

Copy link
Copy Markdown
Member

Landed in 7467a5d

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@indutny@nodejs-github-bot@targos@Trott@cjihrig@trivikr