deps: remove unused foo.js & bar.js from lru-cache#8214
deps: remove unused foo.js & bar.js from lru-cache#8214Fishrock123 wants to merge 1 commit intonpm:masterfrom
Conversation
|
Since it doesn't hurt anything, I rebased this and landed it as 79f8a96. I'll reiterate, though, that the way the npm team deals with bundled dependencies is just to run |
|
Right, but these aren't included in that repo, and the tests/etc don't seem to produce them? |
|
They're in the package as it was published to the registry, hence my suggestion to take it up with the packager (which, in this case, is @isaacs). As much as possible, to keep things moving along, we need to be able to treat |
Ohhhh, right, I didn't check that. Yes, I agree it does not matter in that case, messy or not. Pssst, ya'll should consider using the |
These files are not found in https://github.com/isaacs/node-lru-cache and appear to have been erroneously committed in 530a2e3.
(see also: nodejs/node#1658 (comment))