Skip to content

Commit

Permalink
doc: remove assertions about assert
Browse files Browse the repository at this point in the history
The assert docs have some language that suggests that we don't want bug
fixes. We do. Send in bug fixes, please. (Just no new API features.)
We'd love to not have assert in core at all, but that ship has sailed.
It's here to stay. Let's at least make it not have surprising behaviors.
Because we want good things for our users.

PR-URL: #11113
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
  • Loading branch information
Trott authored and MylesBorins committed Mar 9, 2017
1 parent 89d3090 commit 38938e1
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions doc/api/assert.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,7 @@
> Stability: 2 - Stable
The `assert` module provides a simple set of assertion tests that can be used to
test invariants. The module is intended for internal use by Node.js, but can be
used in application code via `require('assert')`. However, `assert` is not a
testing framework, and is not intended to be used as a general purpose assertion
library.
test invariants.

## assert(value[, message])
<!-- YAML
Expand Down

0 comments on commit 38938e1

Please sign in to comment.