Address some PR comments

Author: hsubox76

tools/gitHooks/api-report.js

@@ -30,7 +30,7 @@ const simpleGit = require('simple-git/promise');
 
 const git = simpleGit(projectRoot);
 
-async function doApiReportsCommit(changedFiles) {
+async function doApiReports(changedFiles) {
   const changedPackages = await getChangedPackages(changedFiles);
   const packageInfo = await getPackageInfo();
   const packageLocations = [];
@@ -64,4 +64,4 @@ async function doApiReportsCommit(changedFiles) {
   });
 }
 
-exports.doApiReportsCommit = doApiReportsCommit;
+exports.doApiReports = doApiReports;

tools/gitHooks/license.js

@@ -76,7 +76,7 @@ function rewriteCopyrightLine(contents) {
   return newLines.join('\n');
 }
 
-async function doLicenseCommit(changedFiles) {
+async function doLicense(changedFiles) {
   const licenseSpinner = ora(' Validating License Headers').start();
 
   const paths = changedFiles.filter(line => line.match(/(js|ts)$/));
@@ -115,7 +115,7 @@ async function doLicenseCommit(changedFiles) {
     symbol: '✅'
   });
 
-  // Diff unstaged (prettier writes) against staged.
+  // Diff unstaged (license writes) against staged.
   const stageDiff = await git.diff(['--name-only']);
 
   if (!stageDiff) {
@@ -136,5 +136,5 @@ async function doLicenseCommit(changedFiles) {
 }
 
 module.exports = {
-  doLicenseCommit
+  doLicense
 };

tools/gitHooks/precommit.js

@@ -15,9 +15,9 @@
  * limitations under the License.
  */
 
-const { doPrettierCommit } = require('./prettier');
-const { doLicenseCommit } = require('./license');
-const { doApiReportsCommit } = require('./api-report');
+const { doPrettier } = require('./prettier');
+const { doLicense } = require('./license');
+const { doApiReports } = require('./api-report');
 const { resolve } = require('path');
 const simpleGit = require('simple-git/promise');
 const ora = require('ora');
@@ -76,13 +76,13 @@ $ git stash pop
     const changedFiles = diff.split('\n');
 
     // Style the code
-    await doPrettierCommit(changedFiles);
+    await doPrettier(changedFiles);
 
     // Validate License headers exist
-    await doLicenseCommit(changedFiles);
+    await doLicense(changedFiles);
 
     // Generate API reports
-    await doApiReportsCommit(changedFiles);
+    await doApiReports(changedFiles);
 
     // Diff staged changes against last commit. Don't do an empty commit.
     const postDiff = await git.diff(['--cached']);

tools/gitHooks/prettier.js

@@ -56,7 +56,7 @@ function checkVersion() {
   });
 }
 
-async function doPrettierCommit(changedFiles) {
+async function doPrettier(changedFiles) {
   try {
     await checkVersion();
   } catch (e) {
@@ -108,5 +108,5 @@ async function doPrettierCommit(changedFiles) {
 }
 
 module.exports = {
-  doPrettierCommit
+  doPrettier
 };