sprintf.js icon indicating copy to clipboard operation
sprintf.js copied to clipboard

Published 20 hours ago • verified publisher icon alexei

Update .npmignore

Open pkuczynski opened this issue 3 years ago • 5 comments

pkuczynski avatar Feb 01 '21 19:02 pkuczynski

See #205 This removes the need for an .npmignore altogether, as it is inclusive, rather than exclusive.

Nytelife26 avatar Feb 23 '21 17:02 Nytelife26

Yeah, it all depends on what one prefers: whitelisting or blacklisting :) There are pros and cons for both.

pkuczynski avatar Feb 24 '21 10:02 pkuczynski

@pkuczynski Ultimately, for bundling, whitelisting is the way to go. If you had to package someone's order for delivery in a warehouse, would you be given a list of things they haven't ordered or a list of things they have? I appreciate the sentiment, though :)

Nytelife26 avatar Feb 24 '21 23:02 Nytelife26

What problem does this address?

alexei avatar Mar 04 '21 08:03 alexei

@alexei It is unnecessary now, due to my PR. However, if you wish to investigate this too, that's fine.

Nytelife26 avatar Mar 04 '21 18:03 Nytelife26