util: remove todo

Most people are going to use the existing option and switching the name now comes with a cost which does not seem to justify the improvement. PR-URL: #24982 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>

Author: BridgeAR

doc/api/util.md

@@ -406,10 +406,6 @@ changes:
  * `showProxy` {boolean} If `true`, then objects and functions that are
  `Proxy` objects will be introspected to show their `target` and `handler`
  objects. **Default:** `false`.
- <!--
- TODO(BridgeAR): Deprecate `maxArrayLength` and replace it with
- `maxEntries`.
- -->
  * `maxArrayLength` {number} Specifies the maximum number of `Array`,
  [`TypedArray`][], [`WeakMap`][] and [`WeakSet`][] elements to include when
  formatting. Set to `null` or `Infinity` to show all elements. Set to `0` or

lib/internal/util/inspect.js

@@ -161,8 +161,6 @@ function inspect(value, opts) {
  colors: inspectDefaultOptions.colors,
  customInspect: inspectDefaultOptions.customInspect,
  showProxy: inspectDefaultOptions.showProxy,
- // TODO(BridgeAR): Deprecate `maxArrayLength` and replace it with
- // `maxEntries`.
  maxArrayLength: inspectDefaultOptions.maxArrayLength,
  breakLength: inspectDefaultOptions.breakLength,
  compact: inspectDefaultOptions.compact,