We should configure @typescript-eslint/consistent-type-imports to enforce type only imports when appropriate.
UPDATE:
Context here is that I received a "Failed to resolve entry for package" error when importing types from a types only package (@deephaven-enterprise/jsapi-types). Since it doesn't have a true entry point, importing things without the type keyword caused the error since there is no index.js. The fix was to use type imports and seemed like a good general rule to force type imports when possible to avoid such scenarios.
We should configure
@typescript-eslint/consistent-type-importsto enforcetypeonly imports when appropriate.UPDATE:
Context here is that I received a "Failed to resolve entry for package" error when importing types from a types only package (@deephaven-enterprise/jsapi-types). Since it doesn't have a true entry point, importing things without the
typekeyword caused the error since there is noindex.js. The fix was to usetypeimports and seemed like a good general rule to forcetypeimports when possible to avoid such scenarios.