Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

Update assert.markdown #25786

Closed
wants to merge 1 commit into from
Closed

Update assert.markdown #25786

wants to merge 1 commit into from

Conversation

daveboivin
Copy link

The current wording "This module is used for writing unit tests for your applications, you can access it with require('assert')." implies that this module should only be used in development while unit testing.

The article "Error Handling in Node.js" by Joyent (https://www.joyent.com/developers/node/design/errors) uses the assert module in an efficient way to validate required function arguments.

The current wording "This module is used for writing unit tests for your applications, you can access it with require('assert')." implies that this module should only be used in development while unit testing.

The article "Error Handling in Node.js" by Joyent (https://www.joyent.com/developers/node/design/errors) uses the assert module in an efficient way to validate required function arguments.
@thefourtheye
Copy link

SGTM

@weifansym
Copy link

Thanks.

At 2015-07-31 06:01:25, "daveboivin" notifications@github.com wrote:

The current wording "This module is used for writing unit tests for your applications, you can access it with require('assert')." implies that this module should only be used in development while unit testing.

The article "Error Handling in Node.js" by Joyent (https://www.joyent.com/developers/node/design/errors) uses the assert module in an efficient way to validate required function arguments.

You can view, comment on, or merge this pull request online at:

#25786

Commit Summary
Update assert.markdown
File Changes
Mdoc/api/assert.markdown (2)
Patch Links:
https://github.com/joyent/node/pull/25786.patch
https://github.com/joyent/node/pull/25786.diff


Reply to this email directly or view it on GitHub.

@jasnell
Copy link
Member

jasnell commented Aug 4, 2015

LGTM. I see this is targeted against master. I will land it both on master and the v0.12 branch.

@daveboivin
Copy link
Author

Thanks everyone.

btw: I've now read the Contributing Documentation so I can follow to the guidelines better next time.

jasnell pushed a commit that referenced this pull request Aug 5, 2015
The current wording "This module is used for writing unit tests for your applications, you can access it with require('assert')." implies that this module should only be used in development while unit testing.

The article "Error Handling in Node.js" by Joyent (https://www.joyent.com/developers/node/design/errors) uses the assert module in an efficient way to validate required function arguments.

Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #25786
@jasnell
Copy link
Member

jasnell commented Aug 5, 2015

Landed in 99414b3. Will cherry pick this into v0.12 as well.

@jasnell jasnell closed this Aug 5, 2015
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.

4 participants