@perfective/eslint-config-react 0.5.0 → 0.6.0-beta

package/index.js

@@ -50,7 +50,9 @@ module.exports = {
         {
             files: ['*.@(spec|test).[jt]s?(x)'],
             rules: {
-                'testing-library/no-unnecessary-act': 'error',
+                'testing-library/no-unnecessary-act': ['error', {
+                        isStrict: true,
+                    }],
             },
         },
     ],

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@perfective/eslint-config-react",
-    "version": "0.5.0",
+    "version": "0.6.0-beta",
     "description": "ESLint shareable rules configuration for React",
     "keywords": ["react", "code quality", "code standard", "code style", "eslint", "eslint-config", "lint", "perfective", "typescript"],
     "author": "Andrey Mikheychik <a.mikheychik@gmail.com>",
@@ -11,14 +11,14 @@
     },
     "license": "MIT",
     "dependencies": {
-        "@perfective/eslint-config": "~0.15.0"
+        "@perfective/eslint-config": "~0.16.0-beta.1"
     },
     "peerDependencies": {
-        "eslint-plugin-jsx-a11y": "^6.4.1",
-        "eslint-plugin-react": "^7.24.0",
-        "eslint-plugin-react-hooks": "^4.2.0",
+        "eslint-plugin-jsx-a11y": "^6.5.1",
+        "eslint-plugin-react": "^7.28.0",
+        "eslint-plugin-react-hooks": "^4.3.0",
         "eslint-plugin-react-hooks-ssr": "^0.1.5",
-        "eslint-plugin-react-perf": "^3.3.0"
+        "eslint-plugin-react-perf": "^3.3.1"
     },
     "main": "./index.js",
     "types": "./index.d.ts",

package/rules/eslint-config/index.js

@@ -10,7 +10,7 @@ module.exports = {
                     'TSTypeParameterInstantiation',
                 ],
             }],
-        '@typescript-eslint/naming-convention': ['error', ...typescript_eslint_1.typescriptEslintNamingConvention()],
-        'unicorn/prevent-abbreviations': ['warn', unicorn_1.unicornPreventAbbreviations()],
+        '@typescript-eslint/naming-convention': ['error', ...(0, typescript_eslint_1.typescriptEslintNamingConvention)()],
+        'unicorn/prevent-abbreviations': ['warn', (0, unicorn_1.unicornPreventAbbreviations)()],
     },
 };

package/rules/eslint-config/rules/typescript-eslint.js

@@ -3,7 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.typescriptEslintNamingConvention = void 0;
 const rules_1 = require("@perfective/eslint-config/rules");
 function typescriptEslintNamingConvention(extensions = []) {
-    return rules_1.typescriptEslintNamingConvention([
+    return (0, rules_1.typescriptEslintNamingConvention)([
         {
             selector: ['variable'],
             modifiers: ['const'],

package/rules/eslint-config/rules/unicorn.js

@@ -3,6 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.unicornPreventAbbreviations = void 0;
 const rules_1 = require("@perfective/eslint-config/rules");
 function unicornPreventAbbreviations(replacements = {}, options = {}) {
-    return rules_1.unicornPreventAbbreviations(Object.assign({ prop: false, props: false, ref: false, refs: false }, replacements), options);
+    return (0, rules_1.unicornPreventAbbreviations)(Object.assign({ prop: false, props: false, ref: false, refs: false }, replacements), options);
 }
 exports.unicornPreventAbbreviations = unicornPreventAbbreviations;

package/rules/react/index.js

@@ -42,7 +42,10 @@ module.exports = {
         'react/no-access-state-in-setstate': 'error',
         'react/no-adjacent-inline-elements': 'error',
         'react/no-array-index-key': 'error',
-        'react/no-children-prop': 'error',
+        'react/no-arrow-function-lifecycle': 'warn',
+        'react/no-children-prop': ['error', {
+                allowFunctions: false,
+            }],
         'react/no-danger': 'error',
         'react/no-danger-with-children': 'error',
         'react/no-deprecated': 'error',
@@ -50,10 +53,12 @@ module.exports = {
         'react/no-did-update-set-state': ['error', 'disallow-in-func'],
         'react/no-direct-mutation-state': 'error',
         'react/no-find-dom-node': 'error',
+        'react/no-invalid-html-attribute': 'warn',
         'react/no-is-mounted': 'error',
         'react/no-multi-comp': ['error', {
                 ignoreStateless: false,
             }],
+        'react/no-namespace': 'error',
         'react/no-redundant-should-component-update': 'error',
         'react/no-render-return-value': 'error',
         'react/no-set-state': 'off',
@@ -74,6 +79,7 @@ module.exports = {
         'react/no-unused-state': 'error',
         'react/no-will-update-set-state': ['error', 'disallow-in-func'],
         'react/prefer-es6-class': ['error', 'always'],
+        'react/prefer-exact-props': 'off',
         'react/prefer-read-only-props': 'warn',
         'react/prefer-stateless-function': 'off',
         'react/prop-types': 'error',

package/rules/react/jsx.js

@@ -76,17 +76,22 @@ module.exports = {
         'react/jsx-no-target-blank': ['warn', {
                 allowReferrer: false,
                 enforceDynamicLinks: 'always',
+                forms: true,
+                links: true,
                 warnOnSpreadAttributes: true,
             }],
         'react/jsx-no-undef': ['error', {
                 allowGlobals: false,
             }],
-        'react/jsx-no-useless-fragment': 'warn',
+        'react/jsx-no-useless-fragment': ['warn', {
+                allowExpressions: true,
+            }],
         'react/jsx-one-expression-per-line': ['warn', {
                 allow: 'single-child',
             }],
         'react/jsx-pascal-case': ['error', {
                 allowAllCaps: false,
+                allowLeadingUnderscore: false,
                 allowNamespace: false,
                 ignore: [],
             }],

package/rules.d.ts

@@ -1,4 +1,3 @@
-import { cypressImportNoExtraneousDependencies, ImportNoExtraneousDependencies, importNoExtraneousDependencies, jestImportNoExtraneousDependencies, jsImportNoExtraneousDependencies, SimpleImportSortImports, simpleImportSortImports, TsLintRule, TsLintRules, TypescriptEslintNamingConvention, TypescriptEslintNamingConventionFormat, TypescriptEslintNamingConventionGroupSelector, TypescriptEslintNamingConventionIndividualSelector, TypescriptEslintNamingConventionSelector, TypescriptEslintNamingConventionUnderscore, TypescriptEslintTslintConfig, typescriptEslintTslintConfig, UnicornPreventAbbreviationReplacements, UnicornPreventAbbreviations } from '@perfective/eslint-config/rules';
-import { typescriptEslintNamingConvention } from './rules/eslint-config/rules/typescript-eslint';
-import { unicornPreventAbbreviations } from './rules/eslint-config/rules/unicorn';
-export { cypressImportNoExtraneousDependencies, ImportNoExtraneousDependencies, importNoExtraneousDependencies, jestImportNoExtraneousDependencies, jsImportNoExtraneousDependencies, SimpleImportSortImports, simpleImportSortImports, TsLintRule, TsLintRules, TypescriptEslintNamingConvention, typescriptEslintNamingConvention, TypescriptEslintNamingConventionFormat, TypescriptEslintNamingConventionGroupSelector, TypescriptEslintNamingConventionIndividualSelector, TypescriptEslintNamingConventionSelector, TypescriptEslintNamingConventionUnderscore, TypescriptEslintTslintConfig, typescriptEslintTslintConfig, UnicornPreventAbbreviationReplacements, UnicornPreventAbbreviations, unicornPreventAbbreviations, };
+export { typescriptEslintNamingConvention } from './rules/eslint-config/rules/typescript-eslint';
+export { unicornPreventAbbreviations } from './rules/eslint-config/rules/unicorn';
+export { cypressImportNoExtraneousDependencies, ImportNoExtraneousDependencies, importNoExtraneousDependencies, jestImportNoExtraneousDependencies, jsImportNoExtraneousDependencies, SimpleImportSortImports, simpleImportSortImports, TsLintRule, TsLintRules, TypescriptEslintNamingConvention, TypescriptEslintNamingConventionFormat, TypescriptEslintNamingConventionGroupSelector, TypescriptEslintNamingConventionIndividualSelector, TypescriptEslintNamingConventionSelector, TypescriptEslintNamingConventionUnderscore, TypescriptEslintTslintConfig, typescriptEslintTslintConfig, UnicornPreventAbbreviationReplacements, UnicornPreventAbbreviations, } from '@perfective/eslint-config/rules';

package/rules.js

@@ -1,14 +1,14 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.unicornPreventAbbreviations = exports.typescriptEslintTslintConfig = exports.typescriptEslintNamingConvention = exports.simpleImportSortImports = exports.jsImportNoExtraneousDependencies = exports.jestImportNoExtraneousDependencies = exports.importNoExtraneousDependencies = exports.cypressImportNoExtraneousDependencies = void 0;
-const rules_1 = require("@perfective/eslint-config/rules");
+exports.typescriptEslintTslintConfig = exports.simpleImportSortImports = exports.jsImportNoExtraneousDependencies = exports.jestImportNoExtraneousDependencies = exports.importNoExtraneousDependencies = exports.cypressImportNoExtraneousDependencies = exports.unicornPreventAbbreviations = exports.typescriptEslintNamingConvention = void 0;
+var typescript_eslint_1 = require("./rules/eslint-config/rules/typescript-eslint");
+Object.defineProperty(exports, "typescriptEslintNamingConvention", { enumerable: true, get: function () { return typescript_eslint_1.typescriptEslintNamingConvention; } });
+var unicorn_1 = require("./rules/eslint-config/rules/unicorn");
+Object.defineProperty(exports, "unicornPreventAbbreviations", { enumerable: true, get: function () { return unicorn_1.unicornPreventAbbreviations; } });
+var rules_1 = require("@perfective/eslint-config/rules");
 Object.defineProperty(exports, "cypressImportNoExtraneousDependencies", { enumerable: true, get: function () { return rules_1.cypressImportNoExtraneousDependencies; } });
 Object.defineProperty(exports, "importNoExtraneousDependencies", { enumerable: true, get: function () { return rules_1.importNoExtraneousDependencies; } });
 Object.defineProperty(exports, "jestImportNoExtraneousDependencies", { enumerable: true, get: function () { return rules_1.jestImportNoExtraneousDependencies; } });
 Object.defineProperty(exports, "jsImportNoExtraneousDependencies", { enumerable: true, get: function () { return rules_1.jsImportNoExtraneousDependencies; } });
 Object.defineProperty(exports, "simpleImportSortImports", { enumerable: true, get: function () { return rules_1.simpleImportSortImports; } });
 Object.defineProperty(exports, "typescriptEslintTslintConfig", { enumerable: true, get: function () { return rules_1.typescriptEslintTslintConfig; } });
-const typescript_eslint_1 = require("./rules/eslint-config/rules/typescript-eslint");
-Object.defineProperty(exports, "typescriptEslintNamingConvention", { enumerable: true, get: function () { return typescript_eslint_1.typescriptEslintNamingConvention; } });
-const unicorn_1 = require("./rules/eslint-config/rules/unicorn");
-Object.defineProperty(exports, "unicornPreventAbbreviations", { enumerable: true, get: function () { return unicorn_1.unicornPreventAbbreviations; } });