rebase, lint & code fixes

Author: sergiolms

package.json

@@ -4593,23 +4593,22 @@
 					"gitlens.advanced.messages": {
 						"type": "object",
 						"default": {
-							"suppressBlameInvalidIgnoreRevsFileBadRevisionWarning": false,
-							"suppressBlameInvalidIgnoreRevsFileWarning": false,
 							"suppressCommitHasNoPreviousCommitWarning": false,
 							"suppressCommitNotFoundWarning": false,
 							"suppressCreatePullRequestPrompt": false,
 							"suppressDebugLoggingWarning": false,
 							"suppressFileNotUnderSourceControlWarning": false,
-							"suppressGitBranchNotFullyMergedWarning": false,
 							"suppressGitDisabledWarning": false,
 							"suppressGitMissingWarning": false,
 							"suppressGitVersionWarning": false,
+							"suppressLineUncommittedWarning": false,
+							"suppressNoRepositoryWarning": false,
+							"suppressRebaseSwitchToTextWarning": false,
 							"suppressIntegrationDisconnectedTooManyFailedRequestsWarning": false,
 							"suppressIntegrationRequestFailed500Warning": false,
 							"suppressIntegrationRequestTimedOutWarning": false,
-							"suppressLineUncommittedWarning": false,
-							"suppressNoRepositoryWarning": false,
-							"suppressRebaseSwitchToTextWarning": false
+							"suppressBlameInvalidIgnoreRevsFileWarning": false,
+							"suppressBlameInvalidIgnoreRevsFileBadRevisionWarning": false
 						},
 						"properties": {
 							"suppressCommitHasNoPreviousCommitWarning": {

src/env/node/git/git.ts

@@ -166,13 +166,11 @@ function getStdinUniqueKey(): number {
 type ExitCodeOnlyGitCommandOptions = GitCommandOptions & { exitCodeOnly: true };
 export type PushForceOptions = { withLease: true; ifIncludes?: boolean } | { withLease: false; ifIncludes?: never };
 
-const branchErrorAndReason = [
+const branchErrorAndReason: [RegExp, BranchErrorReason][] = [
 	[GitErrors.noRemoteReference, BranchErrorReason.NoRemoteReference],
 	[GitErrors.invalidBranchName, BranchErrorReason.InvalidBranchName],
 	[GitErrors.branchAlreadyExists, BranchErrorReason.BranchAlreadyExists],
 	[GitErrors.branchNotFullyMerged, BranchErrorReason.BranchNotFullyMerged],
-	[GitErrors.branchNotYetBorn, BranchErrorReason.BranchNotYetBorn],
-	[GitErrors.branchFastForwardRejected, BranchErrorReason.BranchFastForwardRejected],
 ];
 
 export class Git {

src/env/node/git/localGitProvider.ts

@@ -1255,25 +1255,25 @@ export class LocalGitProvider implements GitProvider, Disposable {
 	}
 
 	@log()
-	createBranch(repoPath: string, name: string, ref: string): Promise<void> {
+	async createBranch(repoPath: string, name: string, ref: string): Promise<void> {
 		try {
-			return void this.git.branch(repoPath, name, ref);
+			await this.git.branch(repoPath, name, ref);
 		} catch (ex) {
 			if (ex instanceof BranchError) {
-				throw ex.WithBranch(branch.name);
+				throw ex.WithBranch(name);
 			}
 
 			throw ex;
 		}
 	}
 
 	@log()
-	renameBranch(repoPath: string, oldName: string, newName: string): Promise<void> {
+	async renameBranch(repoPath: string, oldName: string, newName: string): Promise<void> {
 		try {
-			return void this.git.branch(repoPath, '-m', oldName, newName);
+			await this.git.branch(repoPath, '-m', oldName, newName);
 		} catch (ex) {
 			if (ex instanceof BranchError) {
-				throw ex.WithBranch(branch.name);
+				throw ex.WithBranch(oldName);
 			}
 
 			throw ex;
@@ -1308,7 +1308,7 @@ export class LocalGitProvider implements GitProvider, Disposable {
 			}
 
 			const remote = getRemoteNameFromBranchName(branch.upstream.name);
-			remoteCommit = await this.git.rev_list(repoPath, `refs/remotes/${remote}/${branch.ref}`, {
+			const remoteCommit = await this.git.rev_list(repoPath, `refs/remotes/${remote}/${branch.ref}`, {
 				maxResults: 1,
 			});
 
@@ -1318,8 +1318,8 @@ export class LocalGitProvider implements GitProvider, Disposable {
 				await this.git.branch(repoPath, ...args, branch.ref);
 			} catch (ex) {
 				// If it fails, restore the remote branch
-				await this.git.update_ref(repoPath, `refs/remotes/${remote}/${branch.ref}`, commit);
-				await this.git.branch__set_upstream(repoPath, branch, remote, branch);
+				await this.git.update_ref(repoPath, `refs/remotes/${remote}/${branch.ref}`, remoteCommit?.[0] ?? '');
+				await this.git.branch__set_upstream(repoPath, branch.name, remote, branch.ref);
 				throw ex;
 			}