Commit c237eab
authored
process: remove process.assert
PR-URL: #55035
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>1 parent b64006c commit c237eab
4 files changed
Lines changed: 5 additions & 41 deletions
File tree
- doc/api
- lib/internal
- bootstrap
- process
- test/parallel
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -2142,14 +2142,17 @@ parameter. | |||
| 2142 | 2142 | ||
| 2143 | 2143 | <!-- YAML | |
| 2144 | 2144 | changes: | |
| 2145 | + - version: REPLACEME | ||
| 2146 | + pr-url: https://github.com/nodejs/node/pull/55035 | ||
| 2147 | + description: End-of-Life. | ||
| 2145 | 2148 | - version: v10.0.0 | |
| 2146 | 2149 | pr-url: https://github.com/nodejs/node/pull/18666 | |
| 2147 | 2150 | description: Runtime deprecation. | |
| 2148 | 2151 | - version: v0.3.7 | |
| 2149 | 2152 | description: Documentation-only deprecation. | |
| 2150 | 2153 | --> | |
| 2151 | 2154 | ||
| 2152 | - Type: Runtime | ||
| 2155 | + Type: End-of-Life | ||
| 2153 | 2156 | ||
| 2154 | 2157 | `process.assert()` is deprecated. Please use the [`assert`][] module instead. | |
| 2155 | 2158 | ||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -67,10 +67,7 @@ const { | |||
| 67 | 67 | } = primordials; | |
| 68 | 68 | const config = internalBinding('config'); | |
| 69 | 69 | const internalTimers = require('internal/timers'); | |
| 70 | - const { | ||
| 71 | - defineOperation, | ||
| 72 | - deprecate, | ||
| 73 | - } = require('internal/util'); | ||
| 70 | + const { defineOperation } = require('internal/util'); | ||
| 74 | 71 | const { | |
| 75 | 72 | validateInteger, | |
| 76 | 73 | } = require('internal/validators'); | |
@@ -265,12 +262,6 @@ ObjectDefineProperty(process, 'allowedNodeEnvironmentFlags', { | |||
| 265 | 262 | configurable: true, | |
| 266 | 263 | }); | |
| 267 | 264 | ||
| 268 | - // process.assert | ||
| 269 | - process.assert = deprecate( | ||
| 270 | - perThreadSetup.assert, | ||
| 271 | - 'process.assert() is deprecated. Please use the `assert` module instead.', | ||
| 272 | - 'DEP0100'); | ||
| 273 | - | ||
| 274 | 265 | // TODO(joyeecheung): this property has not been well-maintained, should we | |
| 275 | 266 | // deprecate it in favor of a better API? | |
| 276 | 267 | const { isDebugBuild, hasOpenSSL, hasInspector } = config; | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -33,7 +33,6 @@ const { | |||
| 33 | 33 | const { | |
| 34 | 34 | ErrnoException, | |
| 35 | 35 | codes: { | |
| 36 | - ERR_ASSERTION, | ||
| 37 | 36 | ERR_INVALID_ARG_TYPE, | |
| 38 | 37 | ERR_INVALID_ARG_VALUE, | |
| 39 | 38 | ERR_OUT_OF_RANGE, | |
@@ -53,9 +52,6 @@ let getValidatedPath; // We need to lazy load it because of the circular depende | |||
| 53 | 52 | ||
| 54 | 53 | const kInternal = Symbol('internal properties'); | |
| 55 | 54 | ||
| 56 | - function assert(x, msg) { | ||
| 57 | - if (!x) throw new ERR_ASSERTION(msg || 'assertion error'); | ||
| 58 | - } | ||
| 59 | 55 | const { exitCodes: { kNoFailure } } = internalBinding('errors'); | |
| 60 | 56 | ||
| 61 | 57 | const binding = internalBinding('process_methods'); | |
@@ -428,7 +424,6 @@ const { arch, platform, version } = process; | |||
| 428 | 424 | ||
| 429 | 425 | module.exports = { | |
| 430 | 426 | toggleTraceCategoryState, | |
| 431 | - assert, | ||
| 432 | 427 | buildAllowedFlags, | |
| 433 | 428 | wrapProcessMethods, | |
| 434 | 429 | hrtime, | |
This file was deleted.
0 commit comments