Skip to content

Commit

Permalink
Merge pull request #186 from RightCapitalHQ/renovate/eslint-rule-and-…
Browse files Browse the repository at this point in the history
…plugin-updates

fix(deps): update eslint rule and plugin updates
  • Loading branch information
frantic1048 authored Nov 28, 2024
2 parents daa4dd4 + 425a310 commit 50fce2e
Show file tree
Hide file tree
Showing 8 changed files with 568 additions and 598 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "major",
"comment": "fix(deps): update eslint rule and plugin updates",
"packageName": "@rightcapital/eslint-config",
"email": "29139614+renovate[bot]@users.noreply.github.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "major",
"comment": "fix(deps): update eslint rule and plugin updates",
"packageName": "@rightcapital/eslint-plugin",
"email": "29139614+renovate[bot]@users.noreply.github.com",
"dependentChangeType": "patch"
}
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@
"commitizen": "4.3.1",
"concurrently": "9.0.1",
"eslint": "9.12.0",
"eslint-plugin-eslint-plugin": "6.2.0",
"eslint-plugin-eslint-plugin": "6.3.2",
"execa": "9.5.0",
"husky": "9.1.6",
"prettier": "3.3.3",
Expand Down
18 changes: 9 additions & 9 deletions packages/eslint-config/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,21 +22,21 @@
"prepack": "pnpm run build"
},
"dependencies": {
"@eslint-react/eslint-plugin": "1.9.0",
"@eslint-react/eslint-plugin": "1.17.1",
"@rightcapital/eslint-plugin": "workspace:*",
"@stylistic/eslint-plugin": "2.9.0",
"@typescript-eslint/utils": "8.0.1",
"@stylistic/eslint-plugin": "2.11.0",
"@typescript-eslint/utils": "8.16.0",
"confusing-browser-globals": "1.0.11",
"eslint-import-resolver-typescript": "3.6.1",
"eslint-plugin-import-x": "4.3.1",
"eslint-plugin-jsx-a11y": "6.7.1",
"eslint-import-resolver-typescript": "3.6.3",
"eslint-plugin-import-x": "4.4.3",
"eslint-plugin-jsx-a11y": "6.10.2",
"eslint-plugin-lodash": "8.0.0",
"eslint-plugin-n": "17.10.2",
"eslint-plugin-n": "17.14.0",
"eslint-plugin-react-hooks": "5.0.0",
"eslint-plugin-simple-import-sort": "12.1.1",
"eslint-plugin-unicorn": "56.0.0",
"eslint-plugin-unicorn": "56.0.1",
"globals": "15.11.0",
"typescript-eslint": "8.0.1"
"typescript-eslint": "8.16.0"
},
"devDependencies": {
"@rightcapital/tsconfig": "workspace:*",
Expand Down
3 changes: 3 additions & 0 deletions packages/eslint-config/src/config/mixin/react.ts
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,9 @@ const config: TSESLint.FlatConfig.ConfigArray = [
// https://github.com/facebook/react/blob/1204c789776cb01fbaf3e9f032e7e2ba85a44137/packages/eslint-plugin-react-hooks/src/ExhaustiveDeps.js
'react-hooks/exhaustive-deps': 'error',

// MEMO: There are too many false positives with this rule.
'@eslint-react/hooks-extra/no-direct-set-state-in-use-effect': 'off',

// A11y
// Enforce that all elements that require alternative text have meaningful information
// https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/alt-text.md
Expand Down
6 changes: 3 additions & 3 deletions packages/eslint-plugin/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,14 +25,14 @@
"update:eslint-docs": "pnpm run build && eslint-doc-generator && prettier --write src/rules/**/*.md"
},
"dependencies": {
"@typescript-eslint/utils": "8.0.1"
"@typescript-eslint/utils": "8.16.0"
},
"devDependencies": {
"@rightcapital/tsconfig": "workspace:*",
"@types/eslint": "8.56.12",
"@types/node": "22.8.6",
"@typescript-eslint/parser": "8.0.1",
"@typescript-eslint/rule-tester": "8.0.1",
"@typescript-eslint/parser": "8.16.0",
"@typescript-eslint/rule-tester": "8.16.0",
"@vitest/coverage-v8": "2.1.4",
"@vitest/ui": "2.1.4",
"eslint": "8.57.1",
Expand Down
Loading

0 comments on commit 50fce2e

Please sign in to comment.