fix(git-node): do not assume release commit will conflict

Author: aduh95

lib/promote_release.js

@@ -147,23 +147,31 @@ export default class ReleasePromotion extends Session {
       cli.warn(`Aborting release promotion for version ${version}`);
       throw new Error('Aborted');
     }
-    await this.cherryPickToDefaultBranch();
+    const appliedCleanly = await this.cherryPickToDefaultBranch();
 
-    // Update `node_version.h`
-    await forceRunAsync('git', ['checkout', 'HEAD', '--', 'src/node_version.h'],
+    // Ensure `node_version.h`'s `NODE_VERSION_IS_RELEASE` bit is not updated
+    await forceRunAsync('git',
+      ['checkout', appliedCleanly ? 'HEAD^' : 'HEAD', '--', 'src/node_version.h'],
       { ignoreFailure: false });
 
-    // There will be remaining cherry-pick conflicts the Releaser will
-    // need to resolve, so confirm they've been resolved before
-    // proceeding with next steps.
-    cli.separator();
-    cli.info('Resolve the conflicts and commit the result');
-    cli.separator();
-    const didResolveConflicts = await cli.prompt(
-      'Finished resolving cherry-pick conflicts?', { defaultAnswer: true });
-    if (!didResolveConflicts) {
-      cli.warn(`Aborting release promotion for version ${version}`);
-      throw new Error('Aborted');
+    if (appliedCleanly) {
+      // There were no conflicts, we have to amend the commit to revert the
+      // `node_version.h` changes.
+      await forceRunAsync('git', ['commit', ...this.gpgSign, '--amend', '--no-edit', '-n'],
+        { ignoreFailure: false });
+    } else {
+      // There will be remaining cherry-pick conflicts the Releaser will
+      // need to resolve, so confirm they've been resolved before
+      // proceeding with next steps.
+      cli.separator();
+      cli.info('Resolve the conflicts and commit the result');
+      cli.separator();
+      const didResolveConflicts = await cli.prompt(
+        'Finished resolving cherry-pick conflicts?', { defaultAnswer: true });
+      if (!didResolveConflicts) {
+        cli.warn(`Aborting release promotion for version ${version}`);
+        throw new Error('Aborted');
+      }
     }
 
     if (existsSync('.git/CHERRY_PICK_HEAD')) {
@@ -488,8 +496,14 @@ export default class ReleasePromotion extends Session {
 
     await this.tryResetBranch();
 
-    // There will be conflicts, we do not want to treat this as a failure.
-    await forceRunAsync('git', ['cherry-pick', ...this.gpgSign, releaseCommitSha],
-      { ignoreFailure: true });
+    // There might be conflicts, we do not want to treat this as a hard failure,
+    // but we want to retain that information.
+    try {
+      await forceRunAsync('git', ['cherry-pick', ...this.gpgSign, releaseCommitSha],
+        { ignoreFailure: false });
+      return true;
+    } catch {
+      return false;
+    }
   }
 }