feat(shebang): Add options to ignore unpublished files

[scagood]

This is a possible solution to #119 suggested by @piranna

[scagood]

X_OK: Flag indicating that the file can be executed by the calling process. This has no effect on Windows (will behave like fs.constants.F_OK).
https://nodejs.org/api/fs.html#file-access-constants

Dang it!

I could try to ignore the setting on windows 🤔
Not too sure what to do here 😢

[piranna]

This is ignoring the shebang if the files are in .npmignore, isn't It?

[scagood]

This is ignoring the shebang if the files are in .npmignore, isn't It?

It does not, I can do that though!

[piranna]

I think at least in Linux and macOS would solve the most of use cases.

[scagood]

Yeah, that makes sense. I like it!

[scagood]

I think that is a lot cleaner and quicker too 🤔

[piranna]

I think that is a lot cleaner and quicker too 🤔

Yeah, maybe there would be some other corner cases, but that would be a huge filter anyway.

[aladdin-add]

the code LGTM, can we put it behind an option?

[scagood]

I intend on adding both options this weekend!
Something like:

• additionalExcutables: [ 'tests/**' ]
• ignoreIgnoredFiles: true

[aladdin-add]

LGTM, thanks!