Update Fastify and trim -rc.N suffixes

[jsumners]

Implements the solution proposed by @mcollina in fastify/fastify#763 .

[delvedor]

Should we trim also the version number passed as option to fastify-plugin?

[jsumners]

@delvedor I don't think so:

% cat package.json|jq                                                                                                                                
{
  "dependencies": {
    "fastify": "^1.0.0-rc.1",
    "fastify-plugin": "file:/Users/jsumners/Projects/oss/fastify-plugin"
  }
}

% cat foo.js                                                                                                                                             
'use strict'

const fp = require('fastify-plugin')

const plugin = fp(function (fastify, opts, next) {
  console.log('plugin works')
  next()
}, {
  fastify: '>=1.0.0-rc.1'
})

plugin({}, null, (err) => {
  if (err) throw err
  console.log('done')
})

% node foo.js                                                                                                                                              
plugin works
done

[mcollina]

LGTM

[delvedor]

LGTM

[mcollina]

LGTM

[jsumners]

Well one of you guys will have to merge. I do not have access to this repo.

[mcollina]

done Il giorno ven 9 feb 2018 alle ore 01:13 James Sumners < notifications@github.com> ha scritto:

…

Well one of you guys will have to merge. I do not have access to this repo. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#21 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AADL40QLSyGm9DJOCyUbiXA0S5tEhCwsks5tS42ngaJpZM4R9hsX> .