chore: remove xo in favor of stricter prettier setup (#555)

* chore: remove xo in favor of stricter prettier setup * chore: add prettierignore * chore: restore ava config * style: nicer json formatting
conventional-changelog · Jan 31, 2019 · f24adfc · f24adfc
1 parent dc808de
commit f24adfc
Show file tree

Hide file tree

Showing 42 changed files with 385 additions and 989 deletions.
diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md
@@ -40,7 +40,7 @@ To keep the code base of commitlint neat and tidy the following rules apply to e
 
 > Coding standards
 
-* [Happiness](https://github.com/sindresorhus/xo) enforced
+* `prettier` is king
 * Favor micro library over swiss army knives (rimraf, ncp vs. fs-extra)
 * Be awesome
 

diff --git a/.prettierignore b/.prettierignore
@@ -0,0 +1 @@
+lib/
diff --git a/.vscode/settings.json b/.vscode/settings.json
diff --git a/@alias/commitlint-config-angular/package.json b/@alias/commitlint-config-angular/package.json
@@ -11,7 +11,6 @@
     "deps": "dep-check",
     "pkg": "pkg-check"
   },
-  "xo": false,
   "repository": {
     "type": "git",
     "url": "git+https://github.com/marionebl/commitlint.git"

diff --git a/@alias/commitlint-config-lerna-scopes/package.json b/@alias/commitlint-config-lerna-scopes/package.json
@@ -11,7 +11,6 @@
     "deps": "dep-check",
     "pkg": "pkg-check"
   },
-  "xo": false,
   "repository": {
     "type": "git",
     "url": "git+https://github.com/marionebl/commitlint.git"

diff --git a/@alias/commitlint-config-patternplate/package.json b/@alias/commitlint-config-patternplate/package.json
@@ -11,7 +11,6 @@
     "deps": "dep-check",
     "pkg": "pkg-check"
   },
-  "xo": false,
   "repository": {
     "type": "git",
     "url": "git+https://github.com/marionebl/commitlint.git"

diff --git a/@alias/commitlint/package.json b/@alias/commitlint/package.json
@@ -12,7 +12,6 @@
     "deps": "dep-check",
     "pkg": "pkg-check --skip-main"
   },
-  "xo": false,
   "engines": {
     "node": ">=4"
   },

diff --git a/@commitlint/cli/package.json b/@commitlint/cli/package.json
@@ -15,7 +15,6 @@
     "clean": "npx rimraf lib",
     "deps": "dep-check",
     "pkg": "pkg-check",
-    "lint": "xo",
     "start": "concurrently \"ava -c 4 --verbose --watch\" \"yarn run watch\"",
     "test": "ava -c 4 --verbose",
     "watch": "babel src --out-dir lib --watch --source-maps"
@@ -33,7 +32,6 @@
       "babel-preset-commitlint"
     ]
   },
-  "xo": false,
   "engines": {
     "node": ">=4"
   },
@@ -71,8 +69,7 @@
     "rimraf": "2.6.2",
     "sander": "0.6.0",
     "string-to-stream": "1.1.0",
-    "tmp": "0.0.33",
-    "xo": "0.20.3"
+    "tmp": "0.0.33"
   },
   "dependencies": {
     "@commitlint/format": "^7.4.0",

diff --git a/@commitlint/cli/src/help.js b/@commitlint/cli/src/help.js
@@ -21,12 +21,14 @@ module.exports = flags => {
 			const flags = line[0];
 			const desc = line[1];
 			const defaults = line[2];
-			const fs = flags.map(
-				flag => (flag.length > 1 ? `--${flag}` : ` -${flag}`)
+			const fs = flags.map(flag =>
+				flag.length > 1 ? `--${flag}` : ` -${flag}`
 			);
 			const ds = defaults ? `, defaults to: ${defaults}` : '';
 			const length = flags.reduce((sum, flag) => sum + flag.length, 0);
-			return `${fs.join(',')}${' '.repeat(Math.max(4 + longest - length, 0))}${desc}${ds}`;
+			return `${fs.join(',')}${' '.repeat(
+				Math.max(4 + longest - length, 0)
+			)}${desc}${ds}`;
 		})
 		.join('\n');
 };
diff --git a/@commitlint/config-angular-type-enum/package.json b/@commitlint/config-angular-type-enum/package.json
@@ -9,11 +9,9 @@
     "clean": "exit 0",
     "deps": "dep-check",
     "pkg": "pkg-check",
-    "lint": "xo",
     "start": "exit 0",
     "test": "exit 0"
   },
-  "xo": false,
   "repository": {
     "type": "git",
     "url": "git+https://github.com/marionebl/commitlint.git"
@@ -31,7 +29,6 @@
   },
   "homepage": "https://github.com/marionebl/commitlint#readme",
   "devDependencies": {
-    "@commitlint/utils": "^7.3.1",
-    "xo": "0.20.3"
+    "@commitlint/utils": "^7.3.1"
   }
 }
diff --git a/@commitlint/config-angular/package.json b/@commitlint/config-angular/package.json
@@ -9,11 +9,9 @@
     "clean": "exit 0",
     "deps": "dep-check",
     "pkg": "pkg-check --skip-import",
-    "lint": "xo",
     "start": "exit 0",
     "test": "exit 0"
   },
-  "xo": false,
   "repository": {
     "type": "git",
     "url": "git+https://github.com/marionebl/commitlint.git"

diff --git a/@commitlint/config-conventional/package.json b/@commitlint/config-conventional/package.json
@@ -9,11 +9,9 @@
     "clean": "exit 0",
     "deps": "dep-check",
     "pkg": "pkg-check",
-    "lint": "xo",
     "start": "exit 0",
     "test": "exit 0"
   },
-  "xo": false,
   "publishConfig": {
     "access": "public"
   },

diff --git a/@commitlint/config-lerna-scopes/package.json b/@commitlint/config-lerna-scopes/package.json
@@ -7,13 +7,11 @@
   ],
   "scripts": {
     "clean": "exit 0",
-    "lint": "xo",
     "start": "ava --watch --verbose",
     "test": "ava --verbose",
     "deps": "dep-check",
     "pkg": "pkg-check"
   },
-  "xo": false,
   "ava": {
     "files": [
       "test.js"
@@ -52,7 +50,6 @@
     "@commitlint/utils": "^7.3.1",
     "@lerna/project": "3.5.0",
     "ava": "0.22.0",
-    "lerna": "3.1.1",
-    "xo": "0.20.3"
+    "lerna": "3.1.1"
   }
 }
diff --git a/@commitlint/config-patternplate/package.json b/@commitlint/config-patternplate/package.json
@@ -9,11 +9,9 @@
     "clean": "exit 0",
     "deps": "dep-check",
     "pkg": "pkg-check --skip-import",
-    "lint": "xo",
     "start": "exit 0",
     "test": "exit 0"
   },
-  "xo": false,
   "repository": {
     "type": "git",
     "url": "git+https://github.com/marionebl/commitlint.git"
@@ -36,7 +34,6 @@
     "lodash": "4.17.11"
   },
   "devDependencies": {
-    "@commitlint/utils": "^7.3.1",
-    "xo": "0.20.3"
+    "@commitlint/utils": "^7.3.1"
   }
 }
diff --git a/@commitlint/core/package.json b/@commitlint/core/package.json
@@ -7,10 +7,8 @@
   ],
   "scripts": {
     "deps": "dep-check",
-    "pkg": "pkg-check --skip-import",
-    "lint": "xo"
+    "pkg": "pkg-check --skip-import"
   },
-  "xo": false,
   "engines": {
     "node": ">=4"
   },
@@ -33,9 +31,6 @@
     "email": "[email protected]"
   },
   "license": "MIT",
-  "devDependencies": {
-    "xo": "0.20.3"
-  },
   "dependencies": {
     "@commitlint/format": "^7.4.0",
     "@commitlint/lint": "^7.4.0",

diff --git a/@commitlint/ensure/package.json b/@commitlint/ensure/package.json
@@ -11,7 +11,6 @@
     "clean": "npx rimraf lib",
     "deps": "dep-check",
     "pkg": "pkg-check --skip-import",
-    "lint": "xo",
     "start": "concurrently \"ava -c 4 --verbose --watch\" \"yarn run watch\"",
     "test": "ava -c 4 --verbose",
     "watch": "babel src --out-dir lib --watch --source-maps"
@@ -35,7 +34,6 @@
       "babel-preset-commitlint"
     ]
   },
-  "xo": false,
   "engines": {
     "node": ">=4"
   },
@@ -68,8 +66,7 @@
     "concurrently": "3.5.1",
     "cross-env": "5.1.1",
     "globby": "8.0.1",
-    "rimraf": "2.6.1",
-    "xo": "0.20.3"
+    "rimraf": "2.6.1"
   },
   "dependencies": {
     "lodash": "4.17.11"

diff --git a/@commitlint/ensure/src/case.js b/@commitlint/ensure/src/case.js
@@ -8,10 +8,13 @@ function ensureCase(raw = '', target = 'lowercase') {
 	const input = String(raw)
 		.replace(/`.*?`|".*?"|'.*?'/g, '')
 		.trim();
-	
+
 	const delimiters = /(\/|\\)/g;
-	const transformed = input.split(delimiters)
-		.map(segment => delimiters.test(segment) ? segment : toCase(segment, target))
+	const transformed = input
+		.split(delimiters)
+		.map(segment =>
+			delimiters.test(segment) ? segment : toCase(segment, target)
+		)
 		.join('');
 
 	if (transformed === '' || transformed.match(/^\d/)) {

diff --git a/@commitlint/execute-rule/package.json b/@commitlint/execute-rule/package.json
@@ -11,7 +11,7 @@
     "clean": "npx rimraf lib",
     "deps": "dep-check",
     "pkg": "pkg-check",
-    "lint": "xo",
+
     "start": "concurrently \"ava -c 4 --verbose --watch\" \"yarn run watch\"",
     "test": "ava -c 4 --verbose",
     "watch": "babel src --out-dir lib --watch --source-maps"
@@ -35,7 +35,7 @@
       "babel-preset-commitlint"
     ]
   },
-  "xo": false,
+
   "engines": {
     "node": ">=4"
   },
@@ -68,9 +68,7 @@
     "babel-register": "6.26.0",
     "concurrently": "3.5.1",
     "cross-env": "5.1.1",
-    "rimraf": "2.6.1",
-    "xo": "0.20.3"
-  },
+    "rimraf": "2.6.1"  },
   "dependencies": {
     "babel-runtime": "6.26.0"
   }

diff --git a/@commitlint/format/package.json b/@commitlint/format/package.json
@@ -11,7 +11,7 @@
     "clean": "npx rimraf lib",
     "deps": "dep-check",
     "pkg": "pkg-check --skip-import",
-    "lint": "xo",
+
     "start": "concurrently \"ava -c 4 --verbose --watch\" \"yarn run watch\"",
     "test": "ava -c 4 --verbose",
     "watch": "babel src --out-dir lib --watch --source-maps"
@@ -35,7 +35,7 @@
       "babel-preset-commitlint"
     ]
   },
-  "xo": false,
+
   "engines": {
     "node": ">=4"
   },
@@ -68,9 +68,7 @@
     "concurrently": "3.5.1",
     "cross-env": "5.1.1",
     "lodash": "4.17.11",
-    "rimraf": "2.6.1",
-    "xo": "0.20.3"
-  },
+    "rimraf": "2.6.1"  },
   "dependencies": {
     "babel-runtime": "^6.23.0",
     "chalk": "^2.0.1"

diff --git a/@commitlint/is-ignored/package.json b/@commitlint/is-ignored/package.json
@@ -11,7 +11,7 @@
     "clean": "npx rimraf lib",
     "deps": "dep-check",
     "pkg": "pkg-check",
-    "lint": "xo",
+
     "start": "concurrently \"ava -c 4 --verbose --watch\" \"yarn run watch\"",
     "test": "ava -c 4 --verbose",
     "watch": "babel src --out-dir lib --watch --source-maps"
@@ -35,7 +35,7 @@
       "babel-preset-commitlint"
     ]
   },
-  "xo": false,
+
   "engines": {
     "node": ">=4"
   },
@@ -68,9 +68,7 @@
     "babel-register": "6.26.0",
     "concurrently": "3.5.1",
     "cross-env": "5.1.1",
-    "rimraf": "2.6.1",
-    "xo": "0.20.3"
-  },
+    "rimraf": "2.6.1"  },
   "dependencies": {
     "semver": "5.6.0"
   }

diff --git a/@commitlint/lint/package.json b/@commitlint/lint/package.json
@@ -11,7 +11,7 @@
     "clean": "npx rimraf lib",
     "deps": "dep-check",
     "pkg": "pkg-check --skip-import",
-    "lint": "xo",
+
     "start": "concurrently \"ava -c 4 --verbose --watch\" \"yarn run watch\"",
     "test": "ava -c 4 --verbose",
     "watch": "babel src --out-dir lib --watch --source-maps"
@@ -35,7 +35,7 @@
       "babel-preset-commitlint"
     ]
   },
-  "xo": false,
+
   "engines": {
     "node": ">=4"
   },
@@ -69,9 +69,7 @@
     "cross-env": "5.1.1",
     "execa": "0.9.0",
     "globby": "8.0.1",
-    "rimraf": "2.6.1",
-    "xo": "0.20.3"
-  },
+    "rimraf": "2.6.1"  },
   "dependencies": {
     "@commitlint/is-ignored": "^7.3.1",
     "@commitlint/parse": "^7.3.1",

diff --git a/@commitlint/load/package.json b/@commitlint/load/package.json
@@ -11,7 +11,7 @@
     "clean": "npx rimraf lib",
     "deps": "dep-check",
     "pkg": "pkg-check --skip-import",
-    "lint": "xo",
+
     "start": "concurrently \"ava -c 4 --verbose --watch\" \"yarn run watch\"",
     "test": "ava -c 4 --verbose && ava \"src/*.serial-test.js\" --verbose",
     "watch": "babel src --out-dir lib --watch --source-maps"
@@ -35,7 +35,7 @@
       "babel-preset-commitlint"
     ]
   },
-  "xo": false,
+
   "engines": {
     "node": ">=4"
   },
@@ -69,9 +69,7 @@
     "cross-env": "5.1.1",
     "execa": "0.9.0",
     "globby": "8.0.1",
-    "rimraf": "2.6.1",
-    "xo": "0.20.3"
-  },
+    "rimraf": "2.6.1"  },
   "dependencies": {
     "@commitlint/execute-rule": "^7.3.1",
     "@commitlint/resolve-extends": "^7.3.1",