(work-around) Exclude text file 'text!js-translation.json' from bundling

[tdgroot]

This fixes #47.

Now, I know that this is not the most thorough solution, but this gets my build going again. Feel free to give feedback/merge/decline!

[DrewML]

Thanks @tdgroot! This looks great.

Would you be willing to add a test to cover this? We can add one in computerDepsForBundle.unit.js. I think the test would look roughly like this:

test('js-translation.json not included in computed dep list', () => {
    const graph = {
        a: ['b'],
        b: ['js-translation.json']
    };
    const entries = ['a'];
    expect(computeDepsForBundle(graph, entries)).toEqual(['a', 'b']);
});

[DrewML]

Thoughts on changing this file to something like knownIgnores or excludedFiles? We don't discriminate text vs other resource type in computeDepsForBundle, so we can probably just make use of this file for any future Magento-specific ignores we have.

[tdgroot]

Good point, I renamed the file!

[tdgroot]

Thanks @tdgroot! This looks great.

Would you be willing to add a test to cover this?

Done!

[tdgroot]

Thank you for your review! 👍 I added the requested changes.

[DrewML]

Thanks @tdgroot! Appreciate the contribution 😃