Skip to content

path: replace assertPath() with validator

Rodrigo Muino Tomonari requested to merge github/fork/cjihrig/path into master

The path module's assertPath() does exactly what the validateString() validator does, so this commit updates path to use validateString() instead. A couple drive by updates to validateString() outside of assertPath() are also included.

Technically, the validateString(from, 'from'); and validateString(to, 'to'); changes make the error message different (changes 'path' to 'to'/'from'), but these are already using the new error system, so they shouldn't be considered semver major IMO.

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

Merge request reports