[Fix] order: ignore requires in template literal

[golopot]

Fixes #1936.

[ljharb]

Thanks! It seems like a more general fix is in order; the only requires that this rule should warn on are module-level requires, whether in a template literal or not.

[coveralls]

Coverage increased (+2.5%) to 97.904% when pulling 957092a on golopot:issue-1936 into 6cd1fe1 on benmosher:master.

[golopot]

It sounds great to only warn on module-level requires! There are two cases I am unsure about, they are not strictly module-level requires:

const bar = require('foo').bar

const foo = require('foo')()

I handled both of them anyway, if just for backward compatibility.

[ljharb]

Nice.

[ljharb]

Those are module-level requires, even tho they're immediately destructured. Thanks for handling them!

[mAAdhaTTah]

@golopot Thank you for fixing!