@umijs/lint 4.0.51 → 4.0.53

package/dist/config/eslint/index.js

@@ -13,6 +13,10 @@ var __copyProps = (to, from, except, desc) => {
   return to;
 };
 var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
   isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
   mod
 ));

package/dist/config/eslint/legacy.js

@@ -13,6 +13,10 @@ var __copyProps = (to, from, except, desc) => {
   return to;
 };
 var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
   isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
   mod
 ));

package/dist/config/eslint/rules/fabric.js

@@ -24,10 +24,12 @@ __export(fabric_exports, {
 });
 module.exports = __toCommonJS(fabric_exports);
 var typescript = {
+  // eslint built-in rules
   "no-dupe-class-members": 0,
   "no-redeclare": 0,
   "no-undef": 0,
   "no-unused-vars": 0,
+  // @typescript-eslint rules
   "@typescript-eslint/adjacent-overload-signatures": 0,
   "@typescript-eslint/array-type": 2,
   "@typescript-eslint/await-thenable": 0,
@@ -86,14 +88,17 @@ var typescript = {
   "@typescript-eslint/unified-signatures": 2
 };
 var fabric_default = {
+  // eslint built-in rules
   "no-param-reassign": 2,
   "no-prototype-builtins": 0,
+  // eslint-plugin-react rules
   "react/display-name": 0,
   "react/jsx-key": 1,
   "react/no-array-index-key": 1,
   "react/prop-types": 0,
   "react/react-in-jsx-scope": 0,
   "react/self-closing-comp": 1,
+  // eslint-pluginr-react-hooks rules
   "react-hooks/exhaustive-deps": 1,
   "react-hooks/rules-of-hooks": 2
 };

package/dist/config/eslint/rules/recommended.js

@@ -25,9 +25,13 @@ __export(recommended_exports, {
 });
 module.exports = __toCommonJS(recommended_exports);
 var recommended_default = {
+  // eslint built-in rules
+  // 不需要返回就用 forEach
   "array-callback-return": 2,
+  // eqeq 可能导致潜在的类型转换问题
   eqeqeq: 2,
   "for-direction": 2,
+  // 不加 hasOwnProperty 判断会多出原型链的内容
   "guard-for-in": 2,
   "no-async-promise-executor": 2,
   "no-case-declarations": 2,
@@ -35,12 +39,16 @@ var recommended_default = {
   "no-delete-var": 2,
   "no-dupe-else-if": 2,
   "no-duplicate-case": 2,
+  // eval（）可能导致潜在的安全问题
   "no-eval": 2,
   "no-ex-assign": 2,
   "no-global-assign": 2,
   "no-invalid-regexp": 2,
+  // 没必要改 native 变量
   "no-native-reassign": 2,
+  // 修改对象时，会影响原对象；但是有些场景就是有目的
   "no-param-reassign": 2,
+  // return 值无意义，可能会理解为 resolve
   "no-promise-executor-return": 2,
   "no-self-assign": 2,
   "no-self-compare": 2,
@@ -54,6 +62,8 @@ var recommended_default = {
   "no-with": 2,
   "require-yield": 2,
   "use-isnan": 2,
+  // config-plugin-react rules
+  // button 自带 submit 属性
   "react/button-has-type": 2,
   "react/jsx-key": 2,
   "react/jsx-no-comment-textnodes": 2,
@@ -73,6 +83,7 @@ var recommended_default = {
   "react/no-unescaped-entities": 2,
   "react/no-unknown-property": 2,
   "react/require-render-return": 2,
+  // config-plugin-react-hooks rules
   "react-hooks/rules-of-hooks": 2
 };
 var jest = {
@@ -92,6 +103,7 @@ var jest = {
   "jest/valid-title": 2
 };
 var typescript = {
+  // config-plugin-typescript rules
   "@typescript-eslint/ban-types": 2,
   "@typescript-eslint/no-confusing-non-null-assertion": 2,
   "@typescript-eslint/no-dupe-class-members": 2,

package/dist/config/stylelint/index.js

@@ -13,6 +13,10 @@ var __copyProps = (to, from, except, desc) => {
   return to;
 };
 var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
   isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
   mod
 ));
@@ -32,8 +36,10 @@ module.exports = {
     "function-url-quotes": "always",
     "selector-attribute-quotes": "always",
     "font-family-no-missing-generic-family-keyword": null,
+    // iconfont
     "plugin/declaration-block-no-ignored-properties": true,
     "unit-no-unknown": [true, { ignoreUnits: ["rpx"] }],
+    // webcomponent
     "selector-type-no-unknown": null,
     "value-keyword-case": ["lower", { ignoreProperties: ["composes"] }],
     "selector-class-pattern": [

package/dist/linter/base.js

@@ -17,6 +17,10 @@ var __copyProps = (to, from, except, desc) => {
   return to;
 };
 var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
   isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
   mod
 ));
@@ -32,10 +36,19 @@ var import_child_process = require("child_process");
 var import_path = __toESM(require("path"));
 var BaseLinter = class {
   constructor({ cwd }) {
+    /**
+     * linter package name
+     */
     this.linter = "";
+    /**
+     * paths for linter
+     */
     this.paths = {};
     this.paths.cwd = cwd;
   }
+  /**
+   * get bin file path for current linter
+   */
   getBinPath() {
     try {
       const pkgPath = require.resolve(`${this.linter}/package.json`);
@@ -47,10 +60,16 @@ var BaseLinter = class {
       });
     }
   }
+  /**
+   * get linter fork args
+   */
   getRunArgs(args) {
     args;
     return [];
   }
+  /**
+   * execute linter
+   */
   run(args) {
     (0, import_child_process.fork)(this.getBinPath(), this.getRunArgs(args)).on("exit", (code) => {
       if (!!code) {

package/dist/linter/eslint.js

@@ -17,6 +17,10 @@ var __copyProps = (to, from, except, desc) => {
   return to;
 };
 var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
   isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
   mod
 ));

package/dist/linter/index.js

@@ -17,6 +17,10 @@ var __copyProps = (to, from, except, desc) => {
   return to;
 };
 var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
   isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
   mod
 ));

package/dist/linter/stylelint.js

@@ -17,6 +17,10 @@ var __copyProps = (to, from, except, desc) => {
   return to;
 };
 var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
   isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
   mod
 ));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umijs/lint",
-  "version": "4.0.51",
+  "version": "4.0.53",
   "description": "@umijs/lint",
   "homepage": "https://github.com/umijs/umi/tree/master/packages/lint#readme",
   "bugs": "https://github.com/umijs/umi/issues",
@@ -26,7 +26,7 @@
     "@stylelint/postcss-css-in-js": "^0.38.0",
     "@typescript-eslint/eslint-plugin": "5.48.1",
     "@typescript-eslint/parser": "5.48.1",
-    "@umijs/babel-preset-umi": "4.0.51",
+    "@umijs/babel-preset-umi": "4.0.53",
     "eslint-plugin-jest": "26.1.5",
     "eslint-plugin-react": "7.29.4",
     "eslint-plugin-react-hooks": "4.5.0",