rename deleteBranch to deleteBranches

Author: sergiolms

src/commands/git/branch.ts

@@ -532,7 +532,7 @@ export class BranchGitCommand extends QuickCommand {
 			endSteps(state);
 
 			try {
-				await state.repo.git.deleteBranch(state.references, {
+				await state.repo.git.deleteBranches(state.references, {
 					force: state.flags.includes('--force'),
 					remote: state.flags.includes('--remotes'),
 				});

src/env/node/git/localGitProvider.ts

@@ -1274,7 +1274,7 @@ export class LocalGitProvider implements GitProvider, Disposable {
 	}
 
 	@log()
-	async deleteBranch(
+	async deleteBranches(
 		repoPath: string,
 		branches: GitBranchReference[],
 		options: { force?: boolean; remote?: boolean },
@@ -1307,7 +1307,6 @@ export class LocalGitProvider implements GitProvider, Disposable {
 			}
 		}
 
-		const remoteBranches = branches.filter((b: GitBranchReference) => b.remote);
 		if (remoteBranches.length !== 0) {
 			const branchesByOrigin = groupByMap(remoteBranches, b => getRemoteNameFromBranchName(b.name));
 

src/git/gitProvider.ts

@@ -120,7 +120,7 @@ export interface BranchContributorOverview {
 export interface GitProviderRepository {
 	createBranch?(repoPath: string, name: string, ref: string): Promise<void>;
 	renameBranch?(repoPath: string, oldName: string, newName: string): Promise<void>;
-	deleteBranch?(
+	deleteBranches?(
 		repoPath: string,
 		branches: GitBranchReference | GitBranchReference[],
 		options?: { force?: boolean; remote?: boolean },

src/git/gitProviderService.ts

@@ -1378,19 +1378,19 @@ export class GitProviderService implements Disposable {
 	}
 
 	@log()
-	deleteBranch(
+	deleteBranches(
 		repoPath: string,
 		branches: GitBranchReference | GitBranchReference[],
 		options?: { force?: boolean; remote?: boolean },
 	): Promise<void> {
 		const { provider, path } = this.getProvider(repoPath);
-		if (provider.deleteBranch == null) throw new ProviderNotSupportedError(provider.descriptor.name);
+		if (provider.deleteBranches == null) throw new ProviderNotSupportedError(provider.descriptor.name);
 
 		if (!Array.isArray(branches)) {
 			branches = [branches];
 		}
 
-		return provider.deleteBranch(path, branches, {
+		return provider.deleteBranches(path, branches, {
 			force: options?.force,
 			remote: options?.remote,
 		});