Skip to content

Commit bb22d52

Browse files
committed
refactor: collapse duplicate extraction branches
extractManagedInstallArchive had identical tar xf commands in both the .zip and .tar.xz branches. Collapse to a single call since tar handles both formats with the same arguments. Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
1 parent b58f606 commit bb22d52

1 file changed

Lines changed: 1 addition & 5 deletions

File tree

src/install/managed.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -534,11 +534,7 @@ export async function extractManagedInstallArchive(inputs: ExtractArchiveInputs)
534534
const execCommand = inputs.execCommand ?? defaultExecCommand;
535535
await ensureDir(inputs.destDir);
536536

537-
if (inputs.format === ".zip") {
538-
await execCommand("tar", ["xf", inputs.archivePath, "-C", inputs.destDir], inputs.destDir);
539-
} else {
540-
await execCommand("tar", ["xf", inputs.archivePath, "-C", inputs.destDir], inputs.destDir);
541-
}
537+
await execCommand("tar", ["xf", inputs.archivePath, "-C", inputs.destDir], inputs.destDir);
542538
}
543539

544540
export async function performManagedInstall(inputs: PerformManagedInstallInputs): Promise<ManagedInstallResult> {

0 commit comments

Comments
 (0)