senecajs / seneca-web-adapter-express

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

Allow host application to handle errors #3

Closed tswaters closed 8 years ago

tswaters commented 8 years ago

Fixes #2

(also, some unrelated test refactoring to be consistent with other adapters)

coveralls commented 8 years ago

Coverage Status

Coverage increased (+1.4%) to 93.662% when pulling 766ad85bc941698c2233fd78b5eaefcd5b2bd4ae on tswaters:master into 6497b2b68a906440ae278579ea735563a0602651 on senecajs:master.

mcdonnelldean commented 8 years ago

LGTM. Point of note, don't update the package until you are releasing. It stops issues with people releasing the wrong version number.

tswaters commented 8 years ago

Good call on not updating the package -- I just released patches for all the adapters to include seneca-web@2 as valid peer dependency... the process I find is very error prone and would only get more complicated if the package/changes were updated prior to release.