hapijs / good

hapi process monitoring
Other
525 stars 161 forks source link

updated package to support hapi 8.x.x and 9.x.x #395

Closed leore closed 9 years ago

leore commented 9 years ago

@arb passed the linter

leore commented 9 years ago

@arb fixed. Let me know if there is anything else.

arb commented 9 years ago

Is there anything about this that shouldn't work with hapi10 or 11? Like why the change to <10.x.x?

leore commented 9 years ago

@arb it could. Do you want me to remove the upper limit?

arb commented 9 years ago

Take a look at https://github.com/continuationlabs/tacky/blob/v2.0.4/.travis.yml. Let's see if we can get it tested from version 8-11. We'll drop support for < 10 in a breaking release.

leore commented 9 years ago

@arb done

arb commented 9 years ago

Can you include hapi 11 as well? That came out recently.

leore commented 9 years ago

@arb everything passed. 1 bad build but if you just rebuild it will pass.

leore commented 9 years ago

@arb

leore commented 9 years ago

@arb any update here?

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.