fix: lint during the landing process

Author: codebytere

lib/landing_session.js

@@ -103,8 +103,20 @@ class LandingSession extends Session {
     return command;
   }
 
-  async suggestAfterPatch(patch) {
+  async validateLint() {
     const { cli } = this;
+
+    const linted = await runAsync('make', ['lint']);
+    if (!linted) {
+      cli.warn('There are lint errors in your patch. ' +
+        'Please fix them before proceeding');
+      return false;
+    }
+  }
+
+  async tryCompleteLanding(patch) {
+    const { cli } = this;
+
     const subjects = patch.match(/Subject: \[PATCH.*?\].*/g);
     if (!subjects) {
       cli.warn('Cannot get number of commits in the patch. ' +
@@ -127,6 +139,7 @@ class LandingSession extends Session {
       if (!canFinal) {
         return;
       }
+
       return this.final();
     }
 
@@ -140,16 +153,36 @@ class LandingSession extends Session {
 
   async apply() {
     const { cli } = this;
+
+    // Bail if another landing session is currently in progress.
     if (!this.isApplying()) {
-      cli.warn('This session can not proceed to apply patches, ' +
-        'run `git node land --abort`');
+      cli.warn('Landing session already in progress - ' +
+        'to start a new one run `git node land --abort`');
       return;
     }
     await this.tryResetBranch();
 
     const patch = await this.downloadAndPatch();
+
+    const cleanLint = await this.validateLint(patch);
+    if (!cleanLint) {
+      const correctedLint = await cli.prompt('Corrected lint errors?');
+      if (correctedLint) {
+        await runAsync('git', ['add', '.']);
+
+        // Final message will be edited later - don't try to change it here.
+        await runAsync('git', ['commit', '--amend', '--no-edit']);
+      } else {
+        cli.info('Please fix lint errors and then run ' +
+        '`git node land --amend` followed by ' +
+        '`git node land --continue`.');
+        process.exit(1);
+      }
+    }
+
     this.startAmending();
-    await this.suggestAfterPatch(patch);
+
+    await this.tryCompleteLanding(patch);
   }
 
   async amend() {
@@ -212,7 +245,8 @@ class LandingSession extends Session {
   async final() {
     const { cli, owner, repo, upstream, branch, prid } = this;
 
-    if (!this.readyToFinal()) {  // check git rebase/am has been done
+    // Check that git rebase/am has been completed.
+    if (!this.readyToFinal()) {
       cli.warn('Not yet ready to final');
       cli.log('A git rebase/am is in progress.' +
         ' Please complete it before running git node land --final');
@@ -274,13 +308,14 @@ class LandingSession extends Session {
       return this.amend();
     }
     if (this.isApplying()) {
-      // We are resolving conflict
+      // We're still resolving conflicts.
       if (this.amInProgress()) {
         cli.log('Looks like you are resolving a `git am` conflict');
         cli.log('Please run `git status` for help');
-      } else {  // The conflict has been resolved
+      } else {
+        // Conflicts has been resolved - amend.
         this.startAmending();
-        return this.suggestAfterPatch(this.patch);
+        return this.tryCompleteLanding(this.patch);
       }
       return;
     }