util: move util.isRegExp to eol

marco-ippolito · nodejs-github-bot · commit d3068b9cfa5a · 2024-05-06T09:38:04.000Z
PR-URL: #52744 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md
@@ -1253,6 +1253,9 @@ instead.
 
 <!-- YAML
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/52744
+    description: End-of-Life deprecation.
   - version: v22.0.0
     pr-url: https://github.com/nodejs/node/pull/50488
     description: Runtime deprecation.
@@ -1268,9 +1271,9 @@ changes:
     description: Documentation-only deprecation.
 -->
 
-Type: Runtime
+Type: End-of-Life
 
-The [`util.isRegExp()`][] API is deprecated. Please use
+The `util.isRegExp()` API has been removed. Please use
 `arg instanceof RegExp` instead.
 
 ### DEP0056: `util.isString()`
@@ -3804,7 +3807,6 @@ is deprecated to better align with recommendations per [NIST SP 800-38D][].
 [`util.isFunction()`]: util.md#utilisfunctionobject
 [`util.isObject()`]: util.md#utilisobjectobject
 [`util.isPrimitive()`]: util.md#utilisprimitiveobject
-[`util.isRegExp()`]: util.md#utilisregexpobject
 [`util.log()`]: util.md#utillogstring
 [`util.promisify`]: util.md#utilpromisifyoriginal
 [`util.toUSVString()`]: util.md#utiltousvstringstring
diff --git a/doc/api/util.md b/doc/api/util.md
@@ -3115,31 +3115,6 @@ util.isPrimitive(new Date());
 // Returns: false
 ```
 
-### `util.isRegExp(object)`
-
-<!-- YAML
-added: v0.6.0
-deprecated: v4.0.0
--->
-
-> Stability: 0 - Deprecated
-
-* `object` {any}
-* Returns: {boolean}
-
-Returns `true` if the given `object` is a `RegExp`. Otherwise, returns `false`.
-
-```js
-const util = require('node:util');
-
-util.isRegExp(/some regexp/);
-// Returns: true
-util.isRegExp(new RegExp('another regexp'));
-// Returns: true
-util.isRegExp({});
-// Returns: false
-```
-
 ### `util.log(string)`
 
 <!-- YAML
diff --git a/lib/util.js b/lib/util.js
@@ -363,9 +363,6 @@ module.exports = {
     }
     return internalDeepEqual(a, b);
   },
-  isRegExp: deprecate(types.isRegExp,
-                      'The `util.isRegExp` API is deprecated. Please use `arg instanceof RegExp` instead.',
-                      'DEP0055'),
   isObject: deprecate(isObject,
                       'The `util.isObject` API is deprecated. ' +
                       'Please use `arg !== null && typeof arg === "object"` instead.',
diff --git a/test/parallel/test-util.js b/test/parallel/test-util.js
@@ -40,16 +40,6 @@ assert.strictEqual(util.isArray(/regexp/), false);
 assert.strictEqual(util.isArray(new Error()), false);
 assert.strictEqual(util.isArray({ __proto__: Array.prototype }), false);
 
-// isRegExp
-assert.strictEqual(util.isRegExp(/regexp/), true);
-assert.strictEqual(util.isRegExp(RegExp(), 'foo'), true);
-assert.strictEqual(util.isRegExp(new RegExp()), true);
-assert.strictEqual(util.isRegExp(context('RegExp')()), true);
-assert.strictEqual(util.isRegExp({}), false);
-assert.strictEqual(util.isRegExp([]), false);
-assert.strictEqual(util.isRegExp(new Date()), false);
-assert.strictEqual(util.isRegExp({ __proto__: RegExp.prototype }), false);
-
 // isDate
 assert.strictEqual(util.isDate(new Date()), true);
 assert.strictEqual(util.isDate(new Date(0), 'foo'), true);
