Skip to content

Upgrade joi to v10.1.0 #292

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Upgrade joi to v10.1.0 #292

wants to merge 1 commit into from

Conversation

0xchimz
Copy link

@0xchimz 0xchimz commented Jan 5, 2017

Moment.js was removed from joi since v10.0.0 .... please remove "BIG" module from dependencies. 💃

@ziluvatar
Copy link
Contributor

Similar to #301

After a some investigation, any upgrade of joi from the current version will break compatibility with ES5.
https://github.com/hapijs/joi/commits/master?after=Y3Vyc29yOtCeNCeSeSsaqQE27cSdaZ5UG57jKzI0NA%3D%3D

Let's keep it as is for now, I'll mark it as next major (although I'm not sure if it will be the next one or the following one)

@ziluvatar
Copy link
Contributor

Closing this in favor of: #348

@ziluvatar ziluvatar closed this May 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants