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

Docs: Added documentation for the assert argument #639

Merged
merged 1 commit into from Sep 12, 2016

Conversation

@blacksun1
Copy link
Contributor

blacksun1 commented Sep 10, 2016

Fixes: #634

@@ -447,6 +447,58 @@ If you would like to run a different linter, or even a custom version of eslint
pass the `-n` or `--linter` argument with the path to the lint runner. For example,
if you plan to use jslint, you can install `lab-jslint` then pass `--linter node_modules/lab-jslint`.

## Integration with an assertion library

Using the `--assert` argument allows you to integrate Lab with your favourite assertion library. It works by

This comment has been minimized.

Copy link
@geek

geek Sep 12, 2016

Member

favourite -> favorite

This comment has been minimized.

Copy link
@blacksun1

blacksun1 Sep 12, 2016

Author Contributor

Hey, I am an Aussie living in the UK so, sorry about that. You want me to change it?

This comment has been minimized.

Copy link
@geek

geek Sep 12, 2016

Member

No problem, this is merged and updated!

Thanks for the help.

This comment has been minimized.

Copy link
@geek

geek Sep 12, 2016

Member

Also, I hope I never have to play you in scrabble, seems like an unfair advantage with all of the extra letters :)

@geek

This comment has been minimized.

Copy link
Member

geek commented Sep 12, 2016

@blacksun1 thanks, it looks good, just the minor nit

@geek geek added the documentation label Sep 12, 2016
@geek geek added this to the 11.0.2 milestone Sep 12, 2016
@geek geek merged commit e7a3d6c into hapijs:master Sep 12, 2016
2 checks passed
2 checks passed
Node Security No known vulnerabilities found
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@blacksun1 blacksun1 deleted the blacksun1:issue_634 branch Sep 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.