fix(@angular/cli): fix the test-commit-message script to work with other target branches

vsavkin · hansl · commit 60f0ca04ecd3 · 2017-08-20T11:54:42.000-07:00
diff --git a/scripts/test-commit-messages.js b/scripts/test-commit-messages.js
@@ -31,16 +31,16 @@ logger
     process.exit(1);
   });
 
-// Note: This is based on the gulp task found in the angular/angular repository
-execSync('git fetch origin');
-// Travis doesn't have master when running jobs on other branches (minor/patch/etc).
-execSync('git fetch origin master:master');
-
 // Get PR target branch, default to master for running locally.
 const currentBranch = process.env.TRAVIS_BRANCH
   || process.env.APPVEYOR_REPO_BRANCH
   || 'master';
 
+// Note: This is based on the gulp task found in the angular/angular repository
+execSync('git fetch origin');
+// Travis doesn't have master when running jobs on other branches (minor/patch/etc).
+execSync(`git fetch origin ${currentBranch}:${currentBranch}`);
+
 const output = execSync('git log ' + currentBranch + '..HEAD --reverse --format="%H %s" --no-merges', {
   encoding: 'utf-8'
 });
@@ -52,7 +52,7 @@ if (output.length === 0) {
 const commitsByLine = output.trim().split(/\n/).map(line => {
   return line.trim().split(' ').slice(1).join(' ');
 });
-logger.info(`Examining ${commitsByLine.length} commit(s) between HEAD and master`);
+logger.info(`Examining ${commitsByLine.length} commit(s) between HEAD and ${currentBranch}`);
 
 const someCommitsInvalid = !commitsByLine.every(message => validateCommitMessage(message));
 
