Update no-dynamic-require.md

[yooungt13]

require('../name' + name);
it should not be passed

[coveralls]

Coverage remained the same at 94.86% when pulling d878b36 on yooungt13:patch-1 into e26e898 on benmosher:master.

[ljharb]

this isn't a dynamic require, and it should pass.

[jfmengels]

True, but it seems like it is duplicated a bit below.
@yooungt13 If you could, instead of removing it, replace it by a library import (like lodash), that might be better for the purpose of the example :)

[ljharb]

I thought the purpose was to illustrate that both a string literal, and a template literal with no interpolations, would pass?

[jfmengels]

Aah... yes, you're right. I totally missed the difference in quotes. Yes, this line should stay as it was.

[jfmengels]

Thanks @yooungt13, good catch :)

I'm fine with this, but I left a comment to improve this if you don't mind :)

[jfmengels]

I fixed the issue. Thanks again for reporting this @yooungt13!

[coveralls]

Coverage decreased (-1.3%) to 93.609% when pulling f5b9a37 on yooungt13:patch-1 into e26e898 on benmosher:master.