Recognize require call with template literals (but no placeholders)

[mischnic]

🐛 bug report

require(`lodash`)

should behave just like require("lodash") but are currently ignored.

require(`./${thing}`)

should still be ignored.

💁 Possible Solution

This case should be normalized here:

parcel/packages/transformers/js/src/visitors/dependencies.js

Lines 56 to 61 in a4cf3e9

	types.isIdentifier(callee) &&
	callee.name === 'require' &&
	args.length === 1 &&
	types.isStringLiteral(args[0]) &&
	!hasBinding(ancestors, 'require') &&
	!isInFalsyBranch(ancestors);

and also normalized to "lodash".

Add a test similar to this one:

parcel/packages/core/integration-tests/test/javascript.js

Lines 24 to 35 in 46af247

	it('should produce a basic JS bundle with CommonJS requires', async function() {
	let b = await bundle(
	path.join(__dirname, '/integration/commonjs/index.js'),
	);
	// assert.equal(b.assets.size, 8);
	// assert.equal(b.childBundles.size, 1);
	let output = await run(b);
	assert.equal(typeof output, 'function');
	assert.equal(output(), 3);
	});

[AceTheCreator]

@mischnic I'd love to pick up this issue

[DeMoorJasper]

@alphabet-codes sure, feel free to work on this

[ognjenjevremovic]

Hey @DeMoorJasper is this issue still open and up for grabs?
This seems to me like a welcoming issue for first time contributors to the project.

May I have a chance to provide a MR for this issue?

Many thanks!

[mischnic]

@ognjenjevremovic Sure