fix: add prettier support, mostly via eslint

Author: mscharley

.prettierignore

@@ -71,6 +71,26 @@ common/temp/
 # Prettier-specific overrides
 #-------------------------------------------------------------------------------------------------------------------
 
-/*/*/CHANGELOG.md
-/common/changes
-/common/scripts
+# Rush files
+common/changes/
+common/scripts/
+common/config/
+CHANGELOG.*
+
+# Package manager files
+pnpm-lock.yaml
+yarn.lock
+package-lock.json
+shrinkwrap.json
+
+# Build outputs
+dist
+lib
+
+# Prettier reformats code blocks inside Markdown, which affects rendered output
+*.md
+
+# Projects will manage their own prettier implementations as part of testing
+commands/
+internal/
+presets/

.prettierrc.js

@@ -1,16 +1 @@
-// Documentation for this file: https://prettier.io/docs/en/configuration.html
-module.exports = {
-  // We use a larger print width because Prettier's word-wrapping seems to be tuned
-  // for plain JavaScript without type annotations
-  printWidth: 110,
-
-  // Use .gitattributes to manage newlines
-  endOfLine: 'auto',
-
-  // Use single quotes instead of double quotes
-  singleQuote: true,
-
-  // For ES5, trailing commas cannot be used in function parameters; it is counterintuitive
-  // to use them for arrays only
-  // trailingComma: "none"
-};
+module.exports = require('./presets/prettier');

commands/bootstrap/.npmignore

@@ -0,0 +1 @@
+"@mscharley/prettier-config"

commands/bootstrap/.prettierrc

@@ -7,6 +7,7 @@
   },
   "scripts": {
     "build": "tsc",
+    "precommit": "eslint --ext .js,.jsx,.ts,.tsx --fix src __tests__",
     "test": "npm run test:lint && npm run test:jest",
     "test:lint": "eslint --ext .js,.jsx,.ts,.tsx src __tests__",
     "test:jest": "jest 2>&1"
@@ -29,14 +30,12 @@
     "@babel/preset-env": "~7.9.5",
     "@babel/preset-typescript": "~7.9.0",
     "@mscharley/eslint-config": "workspace:~1.2.9",
+    "@mscharley/prettier-config": "workspace:~1.0.0",
     "@types/jest": "~25.2.1",
     "@types/node": "~13.13.2",
-    "@typescript-eslint/parser": "~4.16.1",
-    "@typescript-eslint/eslint-plugin": "~4.16.1",
     "babel-jest": "~25.4.0",
     "bufferutil": "~4.0.1",
     "canvas": "~2.6.1",
-    "eslint": "~7.21.0",
     "jest": "~25.4.0",
     "typescript": "~4.2.2",
     "utf-8-validate": "~5.0.2"

commands/bootstrap/package.json

@@ -111,6 +111,13 @@
       "summary": "Run tests for all packages.",
       "enableParallelism": false
     },
+    {
+      "commandKind": "bulk",
+      "name": "precommit",
+      "summary": "Run all precommit tests.",
+      "enableParallelism": true,
+      "ignoreDependencyOrder": true
+    },
 
     // {
     //   /**
@@ -140,13 +147,6 @@
       "autoinstallerName": "rush-prettier",
       "shellCommand": "pretty-quick --staged"
     },
-    {
-      "name": "prettier",
-      "commandKind": "global",
-      "summary": "Basic access to prettier globally.",
-      "autoinstallerName": "rush-prettier",
-      "shellCommand": "prettier ."
-    },
     {
       "name": "commitlint",
       "commandKind": "global",
@@ -282,27 +282,6 @@
     //   ]
     // }
 
-    {
-      "parameterKind": "flag",
-      "longName": "--check",
-      "shortName": "-c",
-      "description": "Check if the given files are formatted.",
-      "associatedCommands": ["prettier"]
-    },
-    {
-      "parameterKind": "flag",
-      "longName": "--list-different",
-      "shortName": "-l",
-      "description": "Print the names of files that are different from Prettier's formatting.",
-      "associatedCommands": ["prettier"]
-    },
-    {
-      "parameterKind": "flag",
-      "longName": "--write",
-      "shortName": "-w",
-      "description": "Edit files in-place. (Beware!)",
-      "associatedCommands": ["prettier"]
-    },
     {
       "parameterKind": "string",
       "longName": "--edit",