squash!

This will provide feedback to contributors without getting in the way of writing invalid commit messages locally. The git hook integration is turned off.

Committers who push directly to the repo can be expected to use correct commit messages ;) Most changes go through PRs anyway.

Related to angular#14888

Author: Narretz

package.json

@@ -97,11 +97,12 @@
   },
   "dependencies": {},
   "commitplease": {
-    "style": "angular"
+    "style": "angular",
+    "nohook": true
   },
   "config": {
     "commitizen": {
       "path": "node_modules/cz-conventional-changelog"
     }
   }
-}
+}

scripts/travis/build.sh

@@ -9,7 +9,10 @@ if [ "$JOB" == "ci-checks" ]; then
   grunt ci-checks
   if [[ $TRAVIS_PULL_REQUEST != 'false' ]]; then
     # validate commit messages of all commits in the PR
-    yarn run commitplease -- $TRAVIS_COMMIT_RANGE
+    # convert commit range to 2 dots, as commitplease uses `git log`.
+    # See https://github.com/travis-ci/travis-ci/issues/4596 for more info
+    echo "Validate commit messages in PR."
+    yarn run commitplease -- "${TRAVIS_COMMIT_RANGE/.../..}"
   fi
 elif [ "$JOB" == "unit" ]; then
   if [ "$BROWSER_PROVIDER" == "browserstack" ]; then