chore(ci): fix commitlint (#592)

* chore(dev): added break to the list of type of commit

* chore(project): add commitlint when trying to commit

.cz-config.js

@@ -26,6 +26,7 @@ module.exports = {
 		{ value: "feat", name: "feat:		A new feature" },
 		{ value: "fix", name: "fix:		Bugs, typos, etc" },
 		{ value: "misc", name: "misc:		Other formats like tweaks and such" },
-		{ value: "tests", name: "tests:	Tests, jest, binTestCases, etc" }
+		{ value: "tests", name: "tests:	Tests, jest, binTestCases, etc" },
+		{ value: "break", name: "break: changes that break the behaviour of the cli" }
 	]
 };

commitlint.config.js

@@ -7,23 +7,22 @@
 "use strict";
 
 module.exports = {
-	extends: ["cz", "@commitlint/config-lerna-scopes", "@commitlint/config-angular"],
+	extends: ["cz", "@commitlint/config-lerna-scopes"],
 	rules: {
 		"body-leading-blank": [1, "always"],
-		"body-tense": [1, "always", ["present-imperative"]],
 		"footer-leading-blank": [1, "always"],
-		"footer-tense": [1, "always", ["present-imperative"]],
 		"header-max-length": [2, "always", 80],
 		lang: [0, "always", "eng"],
 		"scope-case": [2, "always", "lowerCase"],
 		"scope-empty": [0, "never"],
-		"subject-case": [1, "always", "lowerCase"],
+		"subject-case": [2, "never", ["sentence-case", "start-case", "pascal-case", "upper-case"]],
 		"subject-empty": [0, "never"],
 		"subject-full-stop": [2, "never", "."],
-		"subject-tense": [1, "always", ["present-imperative"]],
 		"type-case": [2, "always", "lowerCase"],
-		"type-empty": [2, "never"]
-		// The scope-enum :  defined in the cz-config
-		// The 'type-enum':  defined in the cz-config
+		// turn it on once CLI will be inside its own package, so the scope will be the name of the pacakged
+		// part of the @commitlint/config-lerna-scopes
+		"scope-enum": [0, "never"],
+		"type-empty": [2, "never"],
+		"type-enum": [2, "always", ["ast", "break", "chore", "cli", "docs", "feat", "fix", "misc", "tests", "break"]]
 	}
 };

package.json

@@ -35,6 +35,7 @@
     "lint:codeOnly": "eslint \"{bin}/**/!(__testfixtures__)/*.js\" \"{bin}/**.js\"",
     "lint": "eslint \"./bin/*.js\" \"./test/**/*.js\" \"packages/**/!(node_modules)/*.test.js\"",
     "precommit": "lint-staged",
+    "commitmsg": "commitlint -E GIT_PARAMS",
     "pretest": "npm run build && npm run lint && npm run tslint",
     "reportCoverage": "nyc report --reporter=json && codecov -f coverage/coverage-final.json --disable=gcov",
     "test": "nyc jest",
@@ -47,6 +48,10 @@
     "{packages,bin}/**/!(__testfixtures__)/**.js": [
       "eslint --fix",
       "git add"
+    ],
+    "{packages,bin}/**/!(__testfixtures__)/**.ts": [
+      "tslint --fix",
+      "git add"
     ]
   },
   "jest": {
@@ -106,7 +111,6 @@
   },
   "devDependencies": {
     "@commitlint/cli": "^7.1.2",
-    "@commitlint/config-angular": "^7.1.2",
     "@commitlint/config-lerna-scopes": "^7.1.2",
     "@commitlint/prompt-cli": "^7.1.2",
     "@commitlint/travis-cli": "^7.1.2",