@wistia/eslint-config 2.10.0 → 2.10.1

package/.github/workflows/commitlint.yml

@@ -18,7 +18,7 @@ jobs:
     timeout-minutes: 5
     steps:
       - name: Check out Git repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
         with:
           # by default this only fetches the latest commit,
           # but more commits are needed to validate a range of commit messages

package/.github/workflows/lint.yml

@@ -20,7 +20,7 @@ jobs:
     timeout-minutes: 5
     steps:
       - name: Check out Git repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
         with:
           ref: ${{ github.event.pull_request.head.sha || github.ref }}
 

package/.github/workflows/publish-branch-package.yml

@@ -21,7 +21,7 @@ jobs:
     timeout-minutes: 5
     steps:
       - name: Check out Git repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
         with:
           ref: ${{ github.event.inputs.commit_hash || github.event.pull_request.head.sha || github.sha }}
 

package/.github/workflows/release.yml

@@ -31,7 +31,7 @@ jobs:
           repositories: eslint-config
 
       - name: Check out Git repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
         with:
           persist-credentials: false
           # This makes Actions fetch all Git history so that Changesets can generate changelogs with the correct commits
@@ -49,7 +49,7 @@ jobs:
           echo "NEW_PACKAGE_VERSION=$(yarn changeset status --output=release.json && jq -r '.releases[0].newVersion' release.json && rm release.json)" >> "$GITHUB_ENV"
 
       - name: Create Release Pull Request or Publish to npm
-        uses: changesets/action@63a615b9cd06ba9a3e6d13796c7fbcb080a60a0b # v1.8.0
+        uses: changesets/action@a45c4d594aa4e2c509dc14a9f2b3b67ba3780d0d # v1.9.0
         id: changesets
         with:
           createGithubReleases: true

package/.github/workflows/test.yml

@@ -20,7 +20,7 @@ jobs:
     timeout-minutes: 5
     steps:
       - name: Check out Git repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
         with:
           ref: ${{ github.event.pull_request.head.sha || github.ref }}
 

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @wistia/eslint-config
 
+## 2.10.1
+
+### Patch Changes
+
+- [#506](https://github.com/wistia/eslint-config/pull/506) [`9598e0a`](https://github.com/wistia/eslint-config/commit/9598e0aef50271c54e8cd9d078dfc26185cd0c56) Thanks [@okize](https://github.com/okize)! - fix: update `curly` rule to go back to previous setting
+
 ## 2.10.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/eslint-config",
-  "version": "2.10.0",
+  "version": "2.10.1",
   "description": "Wistia's ESLint configurations",
   "packageManager": "yarn@4.14.1",
   "type": "module",
@@ -38,15 +38,15 @@
     "test:export": "check-export-map"
   },
   "dependencies": {
-    "@eslint-react/eslint-plugin": "^5.8.3",
+    "@eslint-react/eslint-plugin": "^5.8.10",
     "@eslint/compat": "^2.1.0",
     "@eslint/js": "^10.0.1",
     "@stylistic/eslint-plugin": "^5.10.0",
-    "@typescript-eslint/eslint-plugin": "^8.59.4",
-    "@typescript-eslint/parser": "^8.59.4",
-    "@vitest/eslint-plugin": "^1.6.17",
+    "@typescript-eslint/eslint-plugin": "^8.60.1",
+    "@typescript-eslint/parser": "^8.60.1",
+    "@vitest/eslint-plugin": "^1.6.19",
     "confusing-browser-globals": "^1.0.11",
-    "eslint-import-resolver-typescript": "^4.4.4",
+    "eslint-import-resolver-typescript": "^4.4.5",
     "eslint-plugin-barrel-files": "^3.0.1",
     "eslint-plugin-filenames": "^1.3.2",
     "eslint-plugin-import-x": "^4.16.2",
@@ -57,9 +57,9 @@
     "eslint-plugin-n": "^18.0.1",
     "eslint-plugin-no-only-tests": "^3.4.0",
     "eslint-plugin-playwright": "^2.10.4",
-    "eslint-plugin-prettier": "^5.5.5",
+    "eslint-plugin-prettier": "^5.5.6",
     "eslint-plugin-promise": "^7.3.0",
-    "eslint-plugin-storybook": "^10.4.0",
+    "eslint-plugin-storybook": "^10.4.2",
     "eslint-plugin-styled-components-a11y": "^2.2.1",
     "eslint-plugin-testing-library": "^7.16.2",
     "globals": "^17.6.0",
@@ -68,16 +68,16 @@
   "devDependencies": {
     "@changesets/changelog-github": "^0.7.0",
     "@changesets/cli": "^2.31.0",
-    "@commitlint/cli": "^21.0.1",
-    "@commitlint/config-conventional": "^21.0.1",
+    "@commitlint/cli": "^21.0.2",
+    "@commitlint/config-conventional": "^21.0.2",
     "check-export-map": "^1.3.1",
-    "eslint": "^10.4.0",
+    "eslint": "^10.4.1",
     "glob": "^13.0.6",
     "husky": "^9.1.7",
     "prettier": "^3.8.3",
-    "storybook": "^10.4.0",
+    "storybook": "^10.4.2",
     "typescript": "^6.0.3",
-    "vitest": "^4.1.7"
+    "vitest": "^4.1.8"
   },
   "peerDependencies": {
     "eslint": ">= 10",

package/src/rules/base.mjs

@@ -322,8 +322,8 @@ export default {
 
   // Enforce consistent brace style for all control statements
   // https://eslint.org/docs/rules/curly
-  // decision: disabled because this is a formatter concern (special case - can be used in some situations)
-  curly: 0,
+  // decision: enforce curly braces for all control statements because even the `multiline` option can allow subtle bugs
+  curly: ['error', 'all'],
 
   // Require default cases in switch statements
   // https://eslint.org/docs/rules/default-case

package/test/__snapshots__/javascript.mjs.snap

@@ -64,7 +64,7 @@
       "undefined": false,
       "unescape": false,
     },
-    "parser": "typescript-eslint/parser@8.59.4",
+    "parser": "typescript-eslint/parser@8.60.1",
     "parserOptions": {
       "requireConfigFile": false,
     },
@@ -80,7 +80,7 @@
     "filenames",
     "import-x:eslint-plugin-import-x@4.16.2",
     "js:@eslint/js@10.0.1",
-    "prettier:eslint-plugin-prettier@5.5.5",
+    "prettier:eslint-plugin-prettier@5.5.6",
     "promise",
   ],
   "rules": {
@@ -737,7 +737,7 @@
       2,
     ],
     "curly": [
-      0,
+      2,
       "all",
     ],
     "default-case": [

package/test/__snapshots__/react.mjs.snap

@@ -1204,7 +1204,7 @@
   },
   "plugins": [
     "@",
-    "@eslint-react:@eslint-react/eslint-plugin@5.8.3",
+    "@eslint-react:@eslint-react/eslint-plugin@5.8.10",
     "jsx-a11y:eslint-plugin-jsx-a11y@6.10.2",
   ],
   "rules": {

package/test/__snapshots__/typescript.mjs.snap

@@ -64,7 +64,7 @@
       "undefined": false,
       "unescape": false,
     },
-    "parser": "typescript-eslint/parser@8.59.4",
+    "parser": "typescript-eslint/parser@8.60.1",
     "parserOptions": {
       "ecmaVersion": 2024,
       "project": "./tsconfig.json",
@@ -78,11 +78,11 @@
   "plugins": [
     "@",
     "@stylistic:@stylistic/eslint-plugin@5.10.0",
-    "@typescript-eslint:@typescript-eslint/eslint-plugin@8.59.4",
+    "@typescript-eslint:@typescript-eslint/eslint-plugin@8.60.1",
     "barrel-files:eslint-plugin-barrel-files@3.0.1",
     "filenames",
     "import-x:eslint-plugin-import-x@4.16.2",
-    "prettier:eslint-plugin-prettier@5.5.5",
+    "prettier:eslint-plugin-prettier@5.5.6",
     "promise",
   ],
   "rules": {
@@ -1174,7 +1174,7 @@
       2,
     ],
     "curly": [
-      0,
+      2,
       "all",
     ],
     "default-case": [