fix: add setup-options for all setup functions

Author: aminya

dist/legacy/setup-cpp.js

@@ -11,7 +11,7 @@ describe("setup-bazel", () => {
     return
   }
   it("should setup bazel", async () => {
-    const installInfo = await setupBazel("", "", process.arch)
+    const installInfo = await setupBazel({ version: "", setupDir: "", arch: process.arch })
 
     await testBin("bazel", ["--version"], (installInfo as InstallationInfo | undefined)?.binDir)
   })

dist/legacy/setup-cpp.js.map

@@ -2,14 +2,14 @@ import { execRoot } from "admina"
 import { hasApk, installApkPack } from "setup-alpine"
 import { addAptKeyViaURL, hasAptGet, installAptPack } from "setup-apt"
 import { installBrewPack } from "setup-brew"
+import type { SetupOptions } from "../setup-options.js"
 import { getDebArch } from "../utils/env/arch.js"
 import { hasDnf } from "../utils/env/hasDnf.js"
 import { isArch } from "../utils/env/isArch.js"
 import { setupChocoPack } from "../utils/setup/setupChocoPack.js"
 import { setupDnfPack } from "../utils/setup/setupDnfPack.js"
 
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
-export async function setupBazel(version: string, _setupDir: string, _arch: string) {
+export async function setupBazel({ version }: SetupOptions) {
   switch (process.platform) {
     case "win32": {
       // install bazelisk because it contains both

dist/modern/setup-cpp.mjs

@@ -5,7 +5,7 @@ import { setupCcache } from "../ccache.js"
 jest.setTimeout(300000)
 describe("setup-ccache", () => {
   it("should setup ccache", async () => {
-    const installInfo = await setupCcache("", "", process.arch)
+    const installInfo = await setupCcache({ version: "", setupDir: "", arch: process.arch })
 
     await testBin("ccache", ["--version"], (installInfo as InstallationInfo | undefined)?.binDir)
   })

dist/modern/setup-cpp.mjs.map

@@ -1,14 +1,14 @@
 import { hasApk, installApkPack } from "setup-alpine"
 import { hasAptGet, installAptPack } from "setup-apt"
 import { installBrewPack } from "setup-brew"
+import type { SetupOptions } from "../setup-options.js"
 import { hasDnf } from "../utils/env/hasDnf.js"
 import { isArch } from "../utils/env/isArch.js"
 import { setupChocoPack } from "../utils/setup/setupChocoPack.js"
 import { setupDnfPack } from "../utils/setup/setupDnfPack.js"
 import { setupPacmanPack } from "../utils/setup/setupPacmanPack.js"
 
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
-export async function setupCcache(version: string, _setupDir: string, _arch: string) {
+export async function setupCcache({ version }: SetupOptions) {
   switch (process.platform) {
     case "win32": {
       return setupChocoPack("ccache", version)

src/bazel/__tests__/bazel.test.ts

@@ -9,7 +9,7 @@ describe("setup-chocolatey", () => {
     return
   }
   it("should setup chocolatey", async () => {
-    const { binDir } = (await setupChocolatey("", "", process.arch)) as InstallationInfo
+    const { binDir } = (await setupChocolatey({ version: "", setupDir: "", arch: process.arch })) as InstallationInfo
     await testBin("choco", ["--version"], binDir)
   })
 })

src/bazel/bazel.ts

@@ -4,19 +4,13 @@ import { execaSync } from "execa"
 import { pathExists } from "path-exists"
 import which from "which"
 import { rcOptions } from "../options.js"
+import type { SetupOptions } from "../setup-options.js"
 import type { InstallationInfo } from "../utils/setup/setupBin.js"
 
 /* eslint-disable require-atomic-updates */
 let binDir: string | undefined
 
-export async function setupChocolatey(
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  _version: string,
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  _setupDir: string,
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  _arch: string,
-): Promise<InstallationInfo | undefined> {
+export async function setupChocolatey(_opts?: SetupOptions): Promise<InstallationInfo | undefined> {
   if (process.platform !== "win32") {
     return undefined
   }