Skip to content

fix a few minor issues flagged by lgtm

Rodrigo Muino Tomonari requested to merge github/fork/rneatherway/lgtm-fixes into master
  • Implicit string concatenation caused the inspector PRESUBMIT to ignore some files
  • Spurious additional passed argument to afterShutdown
  • Confusing (but correct) regex using A-z character range by accident

I'm one of the developers behind https://lgtm.com and I was browsing the nodejs code over there at https://lgtm.com/projects/g/nodejs/node. I found these fairly simple issues that I could fix, hope this is helpful.

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

Merge request reports