nodejs / http-parser

http request/response parser for c
MIT License
6.32k stars 1.53k forks source link

Correct test name and numbering #497

Closed sam-github closed 4 years ago

sam-github commented 4 years ago

It doesn't matter yet, but two tests had the same name, and same test position macro.

sam-github commented 4 years ago

@indutny Who should merge? I have enough github rights to merge, it appears, but I'm not a regular contributor here, so won't merge without an explicit go-ahead.

indutny commented 4 years ago

I don't think we need further review on this PR. Feel free to land it!