@admc.com/eslintplugin-sn-test 3.6.1 → 3.6.2

package/.eslintrc.json

@@ -35,7 +35,7 @@
     "no-eval": "warn",
     "no-extra-label": "error",
     "no-extra-bind": "error",
-    "no-else-return": "error",
+    "no-else-return": ["error", {"allowElseIf": false}],
     "no-array-constructor": "warn",
     "new-cap": "error",
     "max-depth": "warn",
@@ -48,6 +48,13 @@
     "no-constructor-return": "error",
     "no-constant-binary-expression": "error",
     "array-callback-return": ["error", { "checkForEach": true }],
+    "no-unused-vars": ["error", {
+        "args": "all",
+        "argsIgnorePattern": "^_?dummy",
+        "caughtErrors": "all", 
+        "caughtErrorsIgnorePattern": "^_?dummy"
+    }],
+    "no-eq-null": "error",
 
     // Post-ES6
     "no-var": "error",
@@ -63,6 +70,8 @@
     "require-atomic-updates": "error",
     "no-unused-private-class-members": "error",
     "no-promise-executor-return": "error",
+    "prefer-rest-params": "error",
+    "prefer-spread": "error",
 
     // formatting
     "comma-spacing": "warn",

package/.gitattributes

@@ -1,18 +1,18 @@
 * text=auto
 
-*.bash eol=lf
-*.sh eol=lf
-*.ksh eol=lf
-*.csh eol=lf
-*.pem eol=lf
-*.cmd eol=crlf
-*.bat eol=crlf
-*.BAT eol=crlf
-*.CMD eol=crlf
-*.dsp eol=crlf
-*.sln eol=crlf
-*.vcproj eol=crlf
-*.reg eol=crlf
+*.bash text eol=lf
+*.sh text eol=lf
+*.ksh text eol=lf
+*.csh text eol=lf
+*.pem text eol=lf
+*.cmd text eol=crlf
+*.bat text eol=crlf
+*.BAT text eol=crlf
+*.CMD text eol=crlf
+*.dsp text eol=crlf
+*.sln text eol=crlf
+*.vcproj text eol=crlf
+*.reg text eol=crlf
 
 *.PNG -text
 *.png -text
@@ -52,8 +52,8 @@
 *.list text
 *.text text
 *.tpl text
-*.dsv lf
-*.html lf
+*.dsv text eol=lf
+*.html text eol=lf
 *.ent text
 *.xhtml text
 *.dtd text
@@ -97,8 +97,8 @@ makefile text
 *.csv text
 *.shtml text
 *.htm text
-*.js lf
-*.jsm lf
+*.js text eol=lf
+*.jsm text eol=lf
 *.sql text
 *.isq text
 *.nsq text
@@ -112,6 +112,6 @@ makefile text
 *.xsl text
 *.xslt text
 *.dtd text
-*.json lf
+*.json text eol=lf
 *.md text
 LICENSE text

package/bad1Scripts/scoped-es12/sys_ux_data_broker_transform/topLevelArrow.js

@@ -0,0 +1,5 @@
+(iput) => {
+    if (!iput.propertyName) return {"error" : "missing property name"};
+	gs.info(`transformTst:brokertran requested prop '${iput.propertyName}'`);
+	return `${gs.getProperty(iput.propertyName)}>`;
+}

package/goodScripts/all/sys_ux_client_script/arrowFnNoParen.js

@@ -0,0 +1,5 @@
+// Header comment
+a => {
+    console.info(`Something about ${a}`);
+}
+/* Tail comment */

package/goodScripts/all/sys_ux_client_script/arrowNoParam.js

@@ -0,0 +1,5 @@
+// Header comment
+() => {
+    console.info(`Something`);
+}
+/* Tail comment */

package/goodScripts/scoped-es12/sys_ux_data_broker_transform/internalArrow.js

@@ -0,0 +1,5 @@
+function(iput) {
+    if (!iput.propertyNames) return {"error" : "missing property names"};
+    iput.propertyNames.forEach(p => { gs.info(`Next prop (${p})`); });
+    return null;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@admc.com/eslintplugin-sn-test",
-  "version": "3.6.1",
+  "version": "3.6.2",
   "description": "External tests for ESLint plugin @admc.com/eslint-plugin-sn",
   "scripts": {
     "test": "mocha --recursive",

package/sneslintrc.json

@@ -22,6 +22,7 @@
     "eqeqeq": "error",
     "semi": "error",
     "no-extra-parens": "error",
+    "no-eq-null": "error",
     "@admc.com/sn/prefer-array-iterator": "error",
     "@admc.com/sn/no-init-emptystring": "error"
   },
@@ -39,6 +40,8 @@
         "no-useless-rename": "error",
         "no-var": "error",
         "prefer-template": "error",
+        "prefer-rest-params": "error",
+        "prefer-spread": "error",
         "@admc.com/sn/no-client-gr": "error"
       }
     }, {  // ES 12 overrides
@@ -49,6 +52,8 @@
         "object-shorthand": "error",
         "no-useless-rename": "error",
         "no-var": "error",
+        "prefer-rest-params": "error",
+        "prefer-spread": "error",
         "prefer-template": "error"
       }
     }, {  // Overriding for ES12 IIFE scriptlets}