style: Apply prettier config consistently (#2063)

Agoric · Dec 5, 2020 · f836802 · f836802
1 parent 41b51b4
commit f836802
Show file tree

Hide file tree

Showing 39 changed files with 349 additions and 153 deletions.
diff --git a/packages/SwingSet/.prettierrc.json b/packages/SwingSet/.prettierrc.json
diff --git a/packages/SwingSet/package.json b/packages/SwingSet/package.json
@@ -80,9 +80,6 @@
       "@agoric"
     ]
   },
-  "publishConfig": {
-    "access": "public"
-  },
   "ava": {
     "files": [
       "test/**/test-*.js"
@@ -91,5 +88,12 @@
       "esm"
     ],
     "timeout": "2m"
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
+  },
+  "publishConfig": {
+    "access": "public"
   }
 }
diff --git a/packages/acorn-eventual-send/.prettierrc.json b/packages/acorn-eventual-send/.prettierrc.json
diff --git a/packages/acorn-eventual-send/package.json b/packages/acorn-eventual-send/package.json
@@ -48,5 +48,9 @@
     "extends": [
       "@agoric"
     ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   }
 }
diff --git a/packages/agoric-cli/.prettierrc.json b/packages/agoric-cli/.prettierrc.json
diff --git a/packages/agoric-cli/package.json b/packages/agoric-cli/package.json
@@ -55,5 +55,9 @@
     "extends": [
       "@agoric"
     ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   }
 }
diff --git a/packages/bundle-source/.prettierrc.json b/packages/bundle-source/.prettierrc.json
diff --git a/packages/bundle-source/package.json b/packages/bundle-source/package.json
@@ -64,5 +64,9 @@
     "extends": [
       "@agoric"
     ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   }
 }
diff --git a/packages/captp/.prettierrc.json b/packages/captp/.prettierrc.json
diff --git a/packages/captp/package.json b/packages/captp/package.json
@@ -66,5 +66,9 @@
     "extends": [
       "@agoric"
     ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   }
 }
diff --git a/packages/cosmic-swingset/.prettierrc.json b/packages/cosmic-swingset/.prettierrc.json
diff --git a/packages/cosmic-swingset/package.json b/packages/cosmic-swingset/package.json
@@ -77,6 +77,10 @@
       "@agoric"
     ]
   },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
+  },
   "publishConfig": {
     "access": "public"
   },

diff --git a/packages/dapp-svelte-wallet/.prettierrc.json b/packages/dapp-svelte-wallet/.prettierrc.json
diff --git a/packages/dapp-svelte-wallet/api/package.json b/packages/dapp-svelte-wallet/api/package.json
@@ -15,69 +15,32 @@
     "lint-check-jessie": "eslint -c '.eslintrc-jessie.js' '**/*.{js,jsx}'"
   },
   "devDependencies": {
-    "ava": "^3.12.1",
-    "eslint": "^6.1.0",
-    "eslint-config-airbnb-base": "^14.0.0",
-    "eslint-config-jessie": "0.0.3",
-    "eslint-config-prettier": "^6.0.0",
-    "eslint-plugin-import": "^2.18.2",
-    "eslint-plugin-prettier": "^3.1.0",
-    "prettier": "^1.18.2"
+    "@agoric/bundle-source": "^1.1.10",
+    "@agoric/cosmic-swingset": "^0.23.0",
+    "@agoric/eslint-config": "^0.1.0",
+    "@agoric/install-ses": "^0.4.0",
+    "@agoric/registrar": "^0.1.7",
+    "ava": "^3.12.1"
   },
   "dependencies": {
+    "@agoric/assert": "^0.1.0",
     "@agoric/eventual-send": "^0.12.0",
     "@agoric/ertp": "^0.8.0",
+    "@agoric/marshal": "^0.2.7",
     "@agoric/notifier": "^0.2.3",
     "@agoric/promise-kit": "^0.1.7",
     "@agoric/store": "^0.3.1",
+    "@agoric/zoe": "^0.10.0",
     "esm": "^3.2.5"
   },
   "eslintConfig": {
     "extends": [
-      "airbnb-base",
-      "plugin:prettier/recommended",
-      "plugin:jsdoc/recommended"
-    ],
-    "parser": "@typescript-eslint/parser",
-    "env": {
-      "es6": true
-    },
-    "globals": {
-      "harden": "readonly"
-    },
-    "rules": {
-      "implicit-arrow-linebreak": "off",
-      "function-paren-newline": "off",
-      "arrow-parens": "off",
-      "strict": "off",
-      "prefer-destructuring": "off",
-      "no-else-return": "off",
-      "no-console": "off",
-      "no-unused-vars": [
-        "error",
-        {
-          "argsIgnorePattern": "^_",
-          "varsIgnorePattern": "^_"
-        }
-      ],
-      "no-return-assign": "off",
-      "no-param-reassign": "off",
-      "no-restricted-syntax": [
-        "off",
-        "ForOfStatement"
-      ],
-      "no-unused-expressions": "off",
-      "no-loop-func": "off",
-      "no-inner-declarations": "off",
-      "import/prefer-default-export": "off",
-      "import/no-extraneous-dependencies": "off",
-      "jsdoc/no-undefined-types": "off",
-      "jsdoc/require-jsdoc": "off",
-      "jsdoc/require-property-description": "off",
-      "jsdoc/require-param-description": "off",
-      "jsdoc/require-returns": "off",
-      "jsdoc/require-returns-description": "off"
-    }
+      "@agoric"
+    ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   },
   "keywords": [],
   "repository": {

diff --git a/packages/dapp-svelte-wallet/package.json b/packages/dapp-svelte-wallet/package.json
@@ -14,15 +14,7 @@
     ]
   },
   "devDependencies": {
-    "eslint": "^6.8.0",
-    "eslint-config-airbnb": "^18.0.1",
-    "eslint-config-airbnb-base": "^14.0.0",
-    "eslint-config-jessie": "^0.0.6",
-    "eslint-config-prettier": "^6.9.0",
-    "eslint-plugin-import": "^2.20.0",
-    "eslint-plugin-jsx-a11y": "^6.2.3",
-    "eslint-plugin-prettier": "^3.1.2",
-    "prettier": "^1.18.2"
+    "@agoric/eslint-config": "^0.1.0"
   },
   "scripts": {
     "lint-fix": "exit 0",
@@ -43,5 +35,9 @@
     "extends": [
       "@agoric"
     ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   }
 }
diff --git a/packages/deployment/.prettierrc.json b/packages/deployment/.prettierrc.json
diff --git a/packages/deployment/package.json b/packages/deployment/package.json
@@ -24,12 +24,19 @@
     "node-fetch": "^2.6.0",
     "temp": "^0.9.0"
   },
+  "devDependencies": {
+    "@agoric/eslint-config": "^0.1.0"
+  },
   "publishConfig": {
     "access": "public"
   },
   "eslintConfig": {
     "extends": [
       "@agoric"
     ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   }
 }
diff --git a/packages/eslint-config/eslint-config.json b/packages/eslint-config/eslint-config.json
@@ -18,6 +18,15 @@
     "StaticModuleRecord": "readonly"
   },
   "rules": {
+    "quotes": [
+      "error",
+      "single",
+      {
+        "avoidEscape": true,
+        "allowTemplateLiterals": true
+      }
+    ],
+    "comma-dangle": ["error", "always-multiline"],
     "implicit-arrow-linebreak": "off",
     "function-paren-newline": "off",
     "arrow-parens": "off",
@@ -46,7 +55,8 @@
       "error",
       {
         "devDependencies": [
-          "rollup.config.js",
+          "**/*.config.js",
+          "**/*.config.*.js",
           "*test*/**/*.js",
           "scripts/**/*.js"
         ]
@@ -58,9 +68,13 @@
     "jsdoc/require-property-description": "off",
     "jsdoc/require-param-description": "off",
     "jsdoc/require-returns": "off",
-    "jsdoc/require-returns-description": "off"
+    "jsdoc/require-returns-description": "off",
+    "jsdoc/valid-types": "off"
   },
   "ignorePatterns": [
-    "**/dist/**"
+    "**/output/**",
+    "bundles/**",
+    "dist/**",
+    "test262/**"
   ]
 }
diff --git a/packages/eslint-config/package.json b/packages/eslint-config/package.json
@@ -1,8 +1,7 @@
 {
   "name": "@agoric/eslint-config",
-  "version": "0.1.0",
+  "version": "0.1.0+1-dev",
   "description": "Agoric style rules",
-  "private": true,
   "main": "./eslint-config.json",
   "repository": {
     "type": "git",
@@ -27,5 +26,20 @@
   "prettier": {
     "trailingComma": "all",
     "singleQuote": true
+  },
+  "devDependencies": {
+    "@typescript-eslint/parser": "^4.1.0",
+    "eslint": "^6.8.0",
+    "eslint-config-airbnb": "^18.0.1",
+    "eslint-config-airbnb-base": "^14.0.0",
+    "eslint-config-jessie": "^0.0.4",
+    "eslint-config-prettier": "^6.9.0",
+    "eslint-plugin-import": "^2.20.0",
+    "eslint-plugin-jsdoc": "^30.4.2",
+    "eslint-plugin-jsx-a11y": "^6.2.3",
+    "eslint-plugin-prettier": "^3.1.2",
+    "eslint-plugin-react": "^7.21.5",
+    "eslint-plugin-react-hooks": "^4",
+    "prettier": "^1.13.0"
   }
 }
diff --git a/packages/eventual-send/.prettierrc.json b/packages/eventual-send/.prettierrc.json
diff --git a/packages/marshal/.prettierrc.json b/packages/marshal/.prettierrc.json
diff --git a/packages/marshal/package.json b/packages/marshal/package.json
@@ -56,6 +56,10 @@
       "@agoric"
     ]
   },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
+  },
   "publishConfig": {
     "access": "public"
   },

diff --git a/packages/stat-logger/.prettierrc.json b/packages/stat-logger/.prettierrc.json
diff --git a/packages/stat-logger/package.json b/packages/stat-logger/package.json
@@ -41,5 +41,9 @@
     "extends": [
       "@agoric"
     ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   }
 }
diff --git a/packages/swing-store-lmdb/.prettierrc.json b/packages/swing-store-lmdb/.prettierrc.json
diff --git a/packages/swing-store-lmdb/package.json b/packages/swing-store-lmdb/package.json
@@ -42,5 +42,9 @@
     "extends": [
       "@agoric"
     ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   }
 }
diff --git a/packages/swing-store-simple/.prettierrc.json b/packages/swing-store-simple/.prettierrc.json
diff --git a/packages/swing-store-simple/package.json b/packages/swing-store-simple/package.json
@@ -40,5 +40,9 @@
     "extends": [
       "@agoric"
     ]
+  },
+  "prettier": {
+    "trailingComma": "all",
+    "singleQuote": true
   }
 }
diff --git a/packages/swingset-runner/.prettierrc.json b/packages/swingset-runner/.prettierrc.json