[New] no-unused-modules: Add ignoreUnusedTypeExports option

[silverwind]

Fixes: #2694

[silverwind]

Will tweak tests a bit.

[silverwind]

Ok, tests are fine now.

[ljharb]

This seems great, except that it's best when boolean arguments default to false. Can you come up with an alternative name for it so it can default to false?

[silverwind]

This seems great, except that it's best when boolean arguments default to false. Can you come up with an alternative name for it so it can default to false?

Hmm I could invert the name and function but thought that the current name matches nicely into the existing options missingExports and unusedExports.

[silverwind]

Inverted it, so now it's ignoreUnusedTypeExports defaulting to false.

[ljharb]

Your original option name definitely did match better, but i prioritize booleans defaulting to false over option names :-) thanks!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.02%. Comparing base (fc361a9) to head (e96db1a).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3011   +/-   ##
=======================================
  Coverage   96.02%   96.02%           
=======================================
  Files          78       78           
  Lines        3296     3299    +3     
  Branches     1158     1160    +2     
=======================================
+ Hits         3165     3168    +3     
  Misses        131      131           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[silverwind]

This CI failures seem intermittent and the coverage annotation "Added line #L518 was not covered by tests" is incorrect, I verified the line is being hit during tests.