Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Require Joi module in the example of README doc #3

Closed
wants to merge 1 commit into from
Closed

Require Joi module in the example of README doc #3

wants to merge 1 commit into from

Conversation

abbassiddiqi
Copy link

@abbassiddiqi abbassiddiqi commented Jan 28, 2018

Joi is being used in the example, but it is not imported/required

id: Joi.string().min(5).required(),

I checked the package code to see if you are exporting the Joi module, but there only a single function middleware was exported. So I added the require statement for Joi.

@coveralls
Copy link

coveralls commented Jan 28, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling a3b5d6a on abbassiddiqi:require-joi into f044ddf on oxyno-zeta:master.

@abbassiddiqi abbassiddiqi deleted the require-joi branch April 9, 2018 17:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants