feat(core): Add config option to skip files considered for imports

[sauntimo]

Background

• closes [Question] also find dead code only used in tests #136
• When functions are imported into tests, (and depending on the local tsconfig settings), ts-prune may no longer consider them dead code, despite them not being use "in anger" anywhere in the project. In these instances it would be useful to be able to specify a pattern (eg .test.ts) whereby matching source files would be ignored when determining if code is dead.

Changes

• adds -s --skip [regexp] flag which allows users to specify a pattern for source files to ignore
• adds two tests for getPotentiallyUnused to ensure that it's working with and without a skip pattern
• adds a skip pattern case to the integration test suite
• fixes delete command and gitignore for the outfile generated by the integration test, and deletes it from git history.

[nadeesha]

Very nice! Almost there.

Could you add a test case to the integration tests as well?

[sauntimo]

Could you add a test case to the integration tests as well?

Ah yeah, good call. I'll have a look at this in the week. Thanks for your comments, they all make sense to me 👍

[sauntimo]

@nadeesha Hopefully I've addressed your feedback, thanks for reviewing! I just wanted to check about the integration test; ts-prune is only run once and I've add a --skip option to this - is that alright? I figured breaking it out into multiple runs was out of scope for now.

[sauntimo]

@nadeesha hey, sorry I appreciate you're probably busy but I'd love to get this finished off if you get a moment 😁 No worries if you don't get a chance to look at it though. Have a nice weekend!

[nadeesha]

@sauntimo Excellent. I just took a look and almost everything looks great.

I saw however that the integration tests are failing. Could you kindly look into that, and we can merge it right after?

[sauntimo]

Oh, that's embarrassing! Thanks for pointing that out, I'll see if can fix this over the weekend.

[sauntimo]

@nadeesha I think I've fixed the integration test now. I noticed that the generated outfile had somehow gotten git tracked so I updated the command to remove it, and the gitignore which should have stopped it from being tracked. I realised that I had added my case to that outfile rather than the outfile.base 🤦. Hopefully everything is working now!

[github-actions]

🎉 This PR is included in version 0.10.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[nadeesha]

@sauntimo Thanks for addressing the feedback. This is now merged.

[sauntimo]

@nadeesha awesome - thanks for your patience & help with it! ☺️