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

[silverwind]

trafficstars

Fixes: https://github.com/import-js/eslint-plugin-import/issues/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[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

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           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: 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.

[ljharb]

node 22.5 itself is broken rn, so i'll rerun those failing jobs once a fixed 22.5.1 is released.