This repository has been archived by the owner on May 28, 2024. It is now read-only.
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
bbd135d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like Hoek now requires Node 8 which makes this a breaking change for hapi-json-api. It breaks our tests using v2.0.4 (https://travis-ci.org/hoodiehq/hoodie-account-server/jobs/289904308), tests work again with v2.0.3, see hoodiehq/hoodie-account-server#274
I would suggest you revert this change, release v2.0.5 and then release v3.0.0 and officially drop support for Node < 8 if that’s what you want :)
bbd135d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping @kjellmorten
bbd135d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the ping, @gr2m, as I somehow missed your first comment.
And thanks for the suggestion. Your tests should work again with v2.0.5.
bbd135d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks a lot!