search

senecajs / seneca-web-adapter-express

seneca-web adapter for express
MIT License
11 stars 14 forks source link

fix seneca-web dependence #4

Closed gknedo closed 8 years ago

gknedo commented 8 years ago

seneca-web was updated...

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 92.254% when pulling ca5d3c6a5a36820dfd2ce983fd666d20fa553228 on gknedo:master into 6497b2b68a906440ae278579ea735563a0602651 on senecajs:master.

tswaters commented 8 years ago

Hi, thanks for the contribution, good catch.

It is indeed throwing back unmet peer dependency - however, the adapter is technically still valid for seneca-web@1.x - updating it to 2.0 would fail in those cases. I'm going to close this PR and update the peerDependency to match both 1.x and 2.x in each of the adapters.

Thanks again