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

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (3) hide show
  1. package/CHANGELOG.md +11 -0
  2. package/index.js +14 -10
  3. package/package.json +2 -2
package/CHANGELOG.md CHANGED
@@ -3,6 +3,17 @@
3
3
  All notable changes to this project will be documented in this file.
4
4
  See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
5
5
 
6
+ ## [0.95.0](https://github.com/deephaven/web-client-ui/compare/v0.94.0...v0.95.0) (2024-09-20)
7
+
8
+ ### ⚠ BREAKING CHANGES
9
+
10
+ - eslint rule will require type only imports where
11
+ possible
12
+
13
+ ### Code Refactoring
14
+
15
+ - 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)
16
+
6
17
  ## [0.94.0](https://github.com/deephaven/web-client-ui/compare/v0.93.0...v0.94.0) (2024-09-18)
7
18
 
8
19
  **Note:** Version bump only for package @deephaven/eslint-config
package/index.js CHANGED
@@ -89,23 +89,27 @@ module.exports = {
89
89
  plugins: ['@typescript-eslint'],
90
90
  extends: ['plugin:@typescript-eslint/recommended'],
91
91
  rules: {
92
- 'default-case': 'off', // Typescript checks for exhaustive switch/case and doesn't require a default
93
- 'consistent-return': 'off', // JS rule which complains if you don't have a default case
94
- 'no-shadow': 'off',
92
+ '@typescript-eslint/consistent-type-imports': [
93
+ 'error',
94
+ { fixStyle: 'inline-type-imports' },
95
+ ],
96
+ '@typescript-eslint/default-param-last': ['error'],
97
+ '@typescript-eslint/explicit-module-boundary-types': 'error',
98
+ '@typescript-eslint/method-signature-style': 'error',
95
99
  '@typescript-eslint/no-shadow': ['error'],
96
- 'no-use-before-define': 'off',
97
100
  '@typescript-eslint/no-use-before-define': 'error',
98
- 'no-useless-constructor': 'off',
99
- 'react/static-property-placement': ['error', 'static public field'],
101
+ '@typescript-eslint/return-await': 'error',
100
102
  '@typescript-eslint/strict-boolean-expressions': ['error'],
103
+ 'consistent-return': 'off', // JS rule which complains if you don't have a default case
104
+ 'default-case': 'off', // Typescript checks for exhaustive switch/case and doesn't require a default
101
105
  'default-param-last': 'off',
102
- '@typescript-eslint/default-param-last': ['error'],
103
- '@typescript-eslint/explicit-module-boundary-types': 'error',
104
- '@typescript-eslint/method-signature-style': 'error',
105
106
  // `no-return-await` needs to be disabled when enabling `@typescript-eslint/return-await`
106
107
  // to avoid incorrectly reporting errors
107
108
  'no-return-await': 'off',
108
- '@typescript-eslint/return-await': 'error',
109
+ 'no-shadow': 'off',
110
+ 'no-use-before-define': 'off',
111
+ 'no-useless-constructor': 'off',
112
+ 'react/static-property-placement': ['error', 'static public field'],
109
113
  },
110
114
  },
111
115
  {
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@deephaven/eslint-config",
3
- "version": "0.94.1-beta.0+96e3d432",
3
+ "version": "0.95.1-beta.0+ec9b41e9",
4
4
  "description": "Deephaven ESLint configuration",
5
5
  "repository": {
6
6
  "type": "git",
@@ -26,5 +26,5 @@
26
26
  "publishConfig": {
27
27
  "access": "public"
28
28
  },
29
- "gitHead": "96e3d4326a855ae67e65f036a909be130c66c40d"
29
+ "gitHead": "ec9b41e921e2122879f53e9a266084d529664680"
30
30
  }