Skip to content
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

Resolve issue #209, doesn't work with new "joi" package. #210

Merged
merged 6 commits into from
Sep 29, 2020

Conversation

eric-swann-q2
Copy link
Contributor

As described in issue #209 :

This lib has dependencies on @hapi/joi and @hapi/boom.
The @hapi/joi dependency in particular causes a collision when using the new joi library as joi detects two versions and throws an exception.

Solution: Instead set both of the libraries as peer dependencies for maximum flexibility and reference "joi" as opposed to "@hapi/joi" for future versions.

@taina0407
Copy link

@johnbrett any plan for this to merge and release ?

@wy193777
Copy link

This one is the only dependency currently prevent me update to joi.

@johnbrett
Copy link
Owner

This merge was published as v8.0.0. I considered this a breaking change as it required a change in dependencies. Thanks again @eric-swann-q2.

@taina0407 @wy193777 please post here if any issues!

@wy193777
Copy link

Thanks for your immediate response!

@johnbrett
Copy link
Owner

@wy193777 thank @eric-swann-q2, he did all the work here, I just dragged my heels, sorry all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants