CLI: Extend list of ignored ESLint rules for pbjs, fixes protobufjs#1085

johncsnyder · Sep 21, 2019 · d3562d7 · d3562d7
1 parent 18dc7a0
commit d3562d7
Show file tree

Hide file tree

Showing 27 changed files with 28 additions and 28 deletions.
diff --git a/bench/data/static_pbjs.js b/bench/data/static_pbjs.js
@@ -1,4 +1,4 @@
-/*eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins*/
+/*eslint-disable block-scoped-var, id-length, no-control-regex, no-magic-numbers, no-prototype-builtins, no-redeclare, no-shadow, no-var, sort-vars*/
 "use strict";
 
 var $protobuf = require("../../minimal");

diff --git a/cli/pbjs.js b/cli/pbjs.js
@@ -20,7 +20,17 @@ var targets  = util.requireAll("./targets");
  * @returns {number|undefined} Exit code, if known
  */
 exports.main = function main(args, callback) {
-    var lintDefault = "eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins";
+    var lintDefault = "eslint-disable " + [
+        "block-scoped-var",
+        "id-length",
+        "no-control-regex",
+        "no-magic-numbers",
+        "no-prototype-builtins",
+        "no-redeclare",
+        "no-shadow",
+        "no-var",
+        "sort-vars"
+    ].join(", ");
     var argv = minimist(args, {
         alias: {
             target: "t",

diff --git a/cli/pbts.js b/cli/pbts.js
@@ -139,8 +139,7 @@ exports.main = function(args, callback) {
         });
 
         function getImportName(importItem) {
-            var result = path.basename(importItem, ".js")
-            return result.replace(/([-_~.+]\w)/g, match => {
+            return path.basename(importItem, ".js").replace(/([-_~.+]\w)/g, function(match) {
                 return match[1].toUpperCase();
             });
         }

diff --git a/dist/light/protobuf.js b/dist/light/protobuf.js
diff --git a/dist/light/protobuf.min.js b/dist/light/protobuf.min.js
diff --git a/dist/minimal/protobuf.js b/dist/minimal/protobuf.js
diff --git a/dist/minimal/protobuf.min.js b/dist/minimal/protobuf.min.js
diff --git a/dist/protobuf.js b/dist/protobuf.js
diff --git a/dist/protobuf.min.js b/dist/protobuf.min.js
diff --git a/ext/descriptor/index.d.ts b/ext/descriptor/index.d.ts
@@ -1,5 +1,4 @@
 import * as $protobuf from "../..";
-
 export const FileDescriptorSet: $protobuf.Type;
 
 export const FileDescriptorProto: $protobuf.Type;

diff --git a/index.d.ts b/index.d.ts
@@ -507,7 +507,7 @@ export interface IExtensionMapField extends IMapField {
 }
 
 /** Abstract runtime message. */
-export class Message<T extends object = object> {
+export class Message<T extends object> {
 
     /**
      * Constructs a new message instance.

diff --git a/tests/data/comments.d.ts b/tests/data/comments.d.ts
@@ -1,5 +1,4 @@
 import * as $protobuf from "../..";
-
 export interface ITest1 {
     field1?: (string|null);
     field2?: (number|null);

diff --git a/tests/data/comments.js b/tests/data/comments.js
@@ -1,4 +1,4 @@
-/*eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins*/
+/*eslint-disable block-scoped-var, id-length, no-control-regex, no-magic-numbers, no-prototype-builtins, no-redeclare, no-shadow, no-var, sort-vars*/
 "use strict";
 
 var $protobuf = require("../../minimal");

diff --git a/tests/data/convert.d.ts b/tests/data/convert.d.ts
@@ -1,5 +1,4 @@
 import * as $protobuf from "../..";
-
 export interface IMessage {
     stringVal?: (string|null);
     stringRepeated?: (string[]|null);

diff --git a/tests/data/convert.js b/tests/data/convert.js
@@ -1,4 +1,4 @@
-/*eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins*/
+/*eslint-disable block-scoped-var, id-length, no-control-regex, no-magic-numbers, no-prototype-builtins, no-redeclare, no-shadow, no-var, sort-vars*/
 "use strict";
 
 var $protobuf = require("../../minimal");

diff --git a/tests/data/mapbox/vector_tile.d.ts b/tests/data/mapbox/vector_tile.d.ts
@@ -1,5 +1,4 @@
 import * as $protobuf from "../../..";
-
 export namespace vector_tile {
 
     interface ITile {

diff --git a/tests/data/mapbox/vector_tile.js b/tests/data/mapbox/vector_tile.js
@@ -1,4 +1,4 @@
-/*eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins*/
+/*eslint-disable block-scoped-var, id-length, no-control-regex, no-magic-numbers, no-prototype-builtins, no-redeclare, no-shadow, no-var, sort-vars*/
 "use strict";
 
 var $protobuf = require("../../../minimal");

diff --git a/tests/data/package.d.ts b/tests/data/package.d.ts
@@ -1,5 +1,4 @@
 import * as $protobuf from "../..";
-
 export interface IPackage {
     name?: (string|null);
     version?: (string|null);

diff --git a/tests/data/package.js b/tests/data/package.js
@@ -1,4 +1,4 @@
-/*eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins*/
+/*eslint-disable block-scoped-var, id-length, no-control-regex, no-magic-numbers, no-prototype-builtins, no-redeclare, no-shadow, no-var, sort-vars*/
 "use strict";
 
 var $protobuf = require("../../minimal");

diff --git a/tests/data/rpc-es6.d.ts b/tests/data/rpc-es6.d.ts
@@ -1,5 +1,4 @@
 import * as $protobuf from "../..";
-
 export class MyService extends $protobuf.rpc.Service {
     constructor(rpcImpl: $protobuf.RPCImpl, requestDelimited?: boolean, responseDelimited?: boolean);
     public static create(rpcImpl: $protobuf.RPCImpl, requestDelimited?: boolean, responseDelimited?: boolean): MyService;

diff --git a/tests/data/rpc-es6.js b/tests/data/rpc-es6.js
@@ -1,4 +1,4 @@
-/*eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins*/
+/*eslint-disable block-scoped-var, id-length, no-control-regex, no-magic-numbers, no-prototype-builtins, no-redeclare, no-shadow, no-var, sort-vars*/
 import * as $protobuf from "../../minimal";
 
 // Common aliases

diff --git a/tests/data/rpc-reserved.d.ts b/tests/data/rpc-reserved.d.ts
@@ -1,5 +1,4 @@
 import * as $protobuf from "../..";
-
 export class MyService extends $protobuf.rpc.Service {
     constructor(rpcImpl: $protobuf.RPCImpl, requestDelimited?: boolean, responseDelimited?: boolean);
     public static create(rpcImpl: $protobuf.RPCImpl, requestDelimited?: boolean, responseDelimited?: boolean): MyService;

diff --git a/tests/data/rpc-reserved.js b/tests/data/rpc-reserved.js
@@ -1,4 +1,4 @@
-/*eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins*/
+/*eslint-disable block-scoped-var, id-length, no-control-regex, no-magic-numbers, no-prototype-builtins, no-redeclare, no-shadow, no-var, sort-vars*/
 "use strict";
 
 var $protobuf = require("../../minimal");

diff --git a/tests/data/rpc.d.ts b/tests/data/rpc.d.ts
@@ -1,5 +1,4 @@
 import * as $protobuf from "../..";
-
 export class MyService extends $protobuf.rpc.Service {
     constructor(rpcImpl: $protobuf.RPCImpl, requestDelimited?: boolean, responseDelimited?: boolean);
     public static create(rpcImpl: $protobuf.RPCImpl, requestDelimited?: boolean, responseDelimited?: boolean): MyService;

diff --git a/tests/data/rpc.js b/tests/data/rpc.js
@@ -1,4 +1,4 @@
-/*eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins*/
+/*eslint-disable block-scoped-var, id-length, no-control-regex, no-magic-numbers, no-prototype-builtins, no-redeclare, no-shadow, no-var, sort-vars*/
 "use strict";
 
 var $protobuf = require("../../minimal");

diff --git a/tests/data/test.d.ts b/tests/data/test.d.ts
@@ -1,5 +1,4 @@
 import * as $protobuf from "../..";
-
 export namespace jspb {
 
     namespace test {

diff --git a/tests/data/test.js b/tests/data/test.js
@@ -1,4 +1,4 @@
-/*eslint-disable block-scoped-var, no-redeclare, no-control-regex, no-prototype-builtins*/
+/*eslint-disable block-scoped-var, id-length, no-control-regex, no-magic-numbers, no-prototype-builtins, no-redeclare, no-shadow, no-var, sort-vars*/
 "use strict";
 
 var $protobuf = require("../../minimal");