search

insin / package-config-checker

Checks if your dependencies have package.json files config or an .npmignore for packaging
MIT License
47 stars 3 forks source link

Add CLI flag to only output dependencies without files/npmignore #3

Open sindresorhus opened 8 years ago

sindresorhus commented 8 years ago

The ones with ✖. I can use grep, but having a flag would be nicer as I could just get a flat list.

insin commented 8 years ago

It should also print the path to the ones marked with ✖ when using this feature and probably shouldn't use left-spacing when doing do (or this should also be an option which defaults to on when using this new flag).

insin commented 8 years ago

This has been started in PR #7

Any suggested names for this flag? The initial PR uses -b, --bad-only, but given that we know there are lots of false positives and negatives with the checks being performed, and that some people choose to package files you don't need at runtime for their own reasons (see avajs/ava#369) I'd prefer something more neutral :smile:

sindresorhus commented 8 years ago
matheuss commented 8 years ago

I also think that --bad-only is a bad (😜) name, but I couldn't find a better one. +1 for bloated.

matheuss commented 8 years ago

Why should we print the path?