Skip to content

assert: update comments

Rodrigo Muino Tomonari requested to merge github/fork/kaicataldo/update-assert-comments into master

Remove the numbers from the comments to make it clear that assert does not follow the CJS spec. Additionally, clean up the existing comments for consistent formatting/language and ease of reading.

Fixes: https://github.com/nodejs/node/issues/9063

Checklist

Apologies in advance for this, but I've been trying to run the following tests on my machine and keep seeing the same two tests time out. Any guidance on how I might be able to run them locally would be greatly appreciated! This PR only affects JS comments and I was able to run make lint successfully.

  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

assert (though only comments have been changed)

Merge request reports