reportFiles as an exclude alternative

[johnnyreilly]

@Timer said here: #173 (comment)

Ooh, that'd be fantastic to implement. I don't want to ship excludes with CRA (in the tsconfig -- I'd way rather silently filter out tests).

@johnnyreilly could you suggest a location to perform the filtering at?

In the workers themselves, at where the worker aggregates, somewhere else? Can you please point to a file or some lines? 😄

That's a merged PR and so it's less visible. I didn't want it to get lost as I think it's a great suggestion. 😄 Do take a look at the conversation there.

cc @ianschmitz

[johnnyreilly]

Suggested implementation (copied from #173 (comment) )

@Timer - sorry about the delay in replying!

The simplest implementation would seem to be taking the approach ts-loader employs here:

https://github.com/TypeStrong/ts-loader/blob/d49dfbaa7e35c728090554d47a122ad2571cb09c/src/utils.ts#L54-L75

and adding a filter in before the forEach in fork-ts-checker-webpack-plugin here:

https://github.com/Realytics/fork-ts-checker-webpack-plugin/blob/458cd748519c04501298c342de31acf1b2f84336/src/index.ts#L741

This would be handling it in the worker aggregate position. It's possible that there might be a minor performance benefit to doing this in the worker itself but I'm not sure how big the gain would be. Simplicity is a wonderful thing so I reckon we should do it here. (We could always change it later if desired.)