@deephaven/eslint-config 0.94.1-beta.0 → 0.95.1-beta.0

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [0.95.0](https://github.com/deephaven/web-client-ui/compare/v0.94.0...v0.95.0) (2024-09-20)
+
+### ⚠ BREAKING CHANGES
+
+- eslint rule will require type only imports where
+  possible
+
+### Code Refactoring
+
+- Added consistent-type-imports eslint rule and ran --fix ([#2230](https://github.com/deephaven/web-client-ui/issues/2230)) ([2744f97](https://github.com/deephaven/web-client-ui/commit/2744f9793aeac2b70e475a725447dcba1b5f294c)), closes [#2229](https://github.com/deephaven/web-client-ui/issues/2229)
+
 ## [0.94.0](https://github.com/deephaven/web-client-ui/compare/v0.93.0...v0.94.0) (2024-09-18)
 
 **Note:** Version bump only for package @deephaven/eslint-config

package/index.js

@@ -89,23 +89,27 @@ module.exports = {
       plugins: ['@typescript-eslint'],
       extends: ['plugin:@typescript-eslint/recommended'],
       rules: {
-        'default-case': 'off', // Typescript checks for exhaustive switch/case and doesn't require a default
-        'consistent-return': 'off', // JS rule which complains if you don't have a default case
-        'no-shadow': 'off',
+        '@typescript-eslint/consistent-type-imports': [
+          'error',
+          { fixStyle: 'inline-type-imports' },
+        ],
+        '@typescript-eslint/default-param-last': ['error'],
+        '@typescript-eslint/explicit-module-boundary-types': 'error',
+        '@typescript-eslint/method-signature-style': 'error',
         '@typescript-eslint/no-shadow': ['error'],
-        'no-use-before-define': 'off',
         '@typescript-eslint/no-use-before-define': 'error',
-        'no-useless-constructor': 'off',
-        'react/static-property-placement': ['error', 'static public field'],
+        '@typescript-eslint/return-await': 'error',
         '@typescript-eslint/strict-boolean-expressions': ['error'],
+        'consistent-return': 'off', // JS rule which complains if you don't have a default case
+        'default-case': 'off', // Typescript checks for exhaustive switch/case and doesn't require a default
         'default-param-last': 'off',
-        '@typescript-eslint/default-param-last': ['error'],
-        '@typescript-eslint/explicit-module-boundary-types': 'error',
-        '@typescript-eslint/method-signature-style': 'error',
         // `no-return-await` needs to be disabled when enabling `@typescript-eslint/return-await`
         // to avoid incorrectly reporting errors
         'no-return-await': 'off',
-        '@typescript-eslint/return-await': 'error',
+        'no-shadow': 'off',
+        'no-use-before-define': 'off',
+        'no-useless-constructor': 'off',
+        'react/static-property-placement': ['error', 'static public field'],
       },
     },
     {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deephaven/eslint-config",
-  "version": "0.94.1-beta.0+96e3d432",
+  "version": "0.95.1-beta.0+ec9b41e9",
   "description": "Deephaven ESLint configuration",
   "repository": {
     "type": "git",
@@ -26,5 +26,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "96e3d4326a855ae67e65f036a909be130c66c40d"
+  "gitHead": "ec9b41e921e2122879f53e9a266084d529664680"
 }