Skip to content

Add fastify to lookup.json - #586

Merged
targos merged 1 commit into
nodejs:masterfrom
mcollina:add-fastify
Jul 12, 2018
Merged

Add fastify to lookup.json#586
targos merged 1 commit into
nodejs:masterfrom
mcollina:add-fastify

Conversation

@mcollina

@mcollinamcollina commented Jul 3, 2018

Copy link
Copy Markdown
Member

Fastify passes all hard requirements. As for soft requirements, it is
being added because of its HTTP2 support.

Ref: nodejs/node#19453.

cc @delvedor

Checklist
  • contribution guidelines followed here

@mcollina
Add fastify to lookup.json
2cf179b
Fastify passes all hard requirements. As for soft requirements, it is
being added because of its HTTP2 support.
@mcollinamcollina mentioned this pull request Jul 3, 2018
Closed
5 tasks
delvedor

This comment was marked as off-topic.

Sign in to view

targos

This comment was marked as off-topic.

Sign in to view

@targos

targos commented Jul 12, 2018

Copy link
Copy Markdown
Member

CI: https://ci.nodejs.org/blue/organizations/jenkins/citgm-smoker-pipeline/detail/citgm-smoker-pipeline/27/pipeline

@targos
targos merged commit 9cea141 into nodejs:masterJul 12, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@targostargostargos approved these changes
+1 more reviewer
@delvedordelvedordelvedor approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mcollina@targos@delvedor