fix: A files list should not exclude bundled dependencies

[evocateur]

What / Why

This preserves npm v6 behavior when both files and bundleDependencies are configured in the root manifest. While one could workaround this by adding an explicit 'node_modules' to the files list, there is currently no error to indicate this necessity and it yields a frustrating silent error.

References

• Fixes [BUG] v2 does not include bundleDependencies if has files #44

[evocateur]

I guess this was fixed in d9b751e. Would have been nice to be notified.

[isaacs]

@evocateur Sorry, I think I ran into this a few weeks ago myself when working on Arborist.reify, and just fixed the bug without stopping to check if there was a PR for it already.

[evocateur]

@isaacs I completely understand, despite my crabby passive-aggressive comment, for which I apologize. I am reminding myself to work on my empathy, which should have been more evident, especially toward a fellow maintainer.

Thanks for all the work you've been putting into npm lately, it's exciting. :)