Skip to content

lib: extract validateObject validator

Rodrigo Muino Tomonari requested to merge github/fork/zys-contrib/validate-object into master

Pulls out the common argument validateObject validator to internal/validators.

BTW, there is inconsistent ERR_INVALID_ARG_TYPE via the object validation: object or Object, do we need to unify this error message ? Although this is so trivial :)

https://github.com/nodejs/node/blob/951b1c3e09bae8395dde0d4f5cecb1fde117fa94/lib/path.js#L896

https://github.com/nodejs/node/blob/951b1c3e09bae8395dde0d4f5cecb1fde117fa94/lib/child_process.js#L408

Update: unify the error message by lowercase object(should be semver-major?)

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports