Linter: Combine Imports

[Ellpeck]

So, all of the options for this rule kind of suck, so we have to compromise.

• the eslint rule doesn't allow auto-fixing, making fixing hundreds of source files a bit of a pain.
• The import/no-duplicates rule, which we're using now, is fine. However, we'd like to enable the prefer-inline option as well, to allow for type imports and regular imports to be merged. Unfortunately, that behavior is currently broken, so we're not enabling it for now.

It seems like, for the time being, this is the best we can do.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.13%. Comparing base (3cc5277) to head (791f02a).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #701      +/-   ##
==========================================
- Coverage   71.19%   71.13%   -0.07%     
==========================================
  Files         217      217              
  Lines        7047     7032      -15     
  Branches     1095     1095              
==========================================
- Hits         5017     5002      -15     
  Misses       1738     1738              
  Partials      292      292              

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

[EagleoutIce]

Thank you!

Unfortunately, that behavior import-js/eslint-plugin-import#2792, so we're not enabling it for now.

Can you create an issue for this?

[EagleoutIce]

This pull request is included in v1.4.2 (see Release v1.4.2 (Dropping xmlparsedata, Benchmark Re-Runs, and Repl Fixes)).