Skip to content

doc: fix more type case inconsistencies

Rodrigo Muino Tomonari requested to merge github/fork/silverwind/type-fixes-part-2 into master
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

Turns out I missed quite a few types in https://github.com/nodejs/node/commit/ff1361957df58ef92c8a9dfce40c6091ea3a21d4 because my sed skills weren't up to par. This commit should fix all remaining issues with primitive type cases.

I also made the syntax with multiple types consistent by removing any whitespace between curly braces. If more people prefer {type | type | type} over {type|type|type}, I can also change to that.

Type replacement was done with:

sed -i -E 's#(\{|\|)\s+?(Boolean|Null|Undefined|Number|String|Symbol)\s+?(\||\})#\1\L\2\3#g' *.md

Merge request reports