update dist folder

SAP · Jan 29, 2025 · 13c16ae · 13c16ae
1 parent fb21050
commit 13c16ae
Show file tree

Hide file tree

Showing 7 changed files with 35 additions and 48 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/config.ts b/src/config.ts
@@ -214,7 +214,6 @@ export async function downloadStageConfig (actionCfg: ActionConfiguration): Prom
     stageConfigPath = actionCfg.customStageConditionsPath
   } else {
     info('using default stage conditions')
-    // TODO: maybe get it from stage-config.json ?
     stageConfigPath = await getEnterpriseConfigUrl(
       STAGE_CONFIG,
       actionCfg.gitHubEnterpriseApi,

diff --git a/src/docker.ts b/src/docker.ts
@@ -18,9 +18,7 @@ export async function runContainers (actionCfg: ActionConfiguration, ctxConfig:
 
 export async function startContainer (actionCfg: ActionConfiguration, ctxConfig: any): Promise<void> {
   const dockerImage = actionCfg.dockerImage !== '' ? actionCfg.dockerImage : ctxConfig.dockerImage
-  if (dockerImage === undefined || dockerImage === '') {
-    return
-  }
+  if (dockerImage === undefined || dockerImage === '') return
 
   const piperPath = internalActionVariables.piperBinPath
   const containerID = uuidv4()

diff --git a/src/enterprise.ts b/src/enterprise.ts
@@ -23,9 +23,7 @@ export function onGitHubEnterprise (): boolean {
 
 export async function getEnterpriseConfigUrl (configType: string, apiURL: string, version: string, token: string, owner: string, repository: string): Promise<string> {
   debug('Getting enterprise config URL')
-  if (configType !== DEFAULT_CONFIG && configType !== STAGE_CONFIG) {
-    return ''
-  }
+  if (configType !== DEFAULT_CONFIG && configType !== STAGE_CONFIG) return ''
 
   debug('initiating assetName and filename')
   let assetName: string = ENTERPRISE_DEFAULTS_FILENAME_ON_RELEASE
@@ -37,7 +35,6 @@ export async function getEnterpriseConfigUrl (configType: string, apiURL: string
     filename = ENTERPRISE_STAGE_CONFIG_FILENAME
   }
 
-  debug('Getting enterprise config URL')
   // if version starts with devel: then it should use inner source Piper
   if (version.startsWith('devel:')) {
     debug(`version starts with "devel:" => ${version}`)

diff --git a/src/execute.ts b/src/execute.ts
@@ -51,21 +51,20 @@ export async function executePiper (
       .catch(err => {
         throw new Error(`Piper execution error: ${err as string}: ${piperError}`)
       })
-  } else {
-    return await exec('docker', [
-      'exec',
-      containerID,
+  }
+  return await exec('docker', [
+    'exec',
+    containerID,
       `/piper/${path.basename(piperPath)}`,
       stepName,
       ...flags
-    ], options).then(exitCode => {
-      return {
-        output: piperOutput,
-        error: piperError,
-        exitCode
-      }
-    }).catch(err => {
-      throw new Error(`Piper execution error: ${err as string}: ${piperError}`)
-    })
-  }
+  ], options).then(exitCode => {
+    return {
+      output: piperOutput,
+      error: piperError,
+      exitCode
+    }
+  }).catch(err => {
+    throw new Error(`Piper execution error: ${err as string}: ${piperError}`)
+  })
 }
diff --git a/src/fetch.ts b/src/fetch.ts
@@ -27,7 +27,7 @@ export async function fetchRetry (url: string, method = 'GET', tries = 5, baseDe
       await wait(delayTime)
     }
   }
-  return await Promise.reject(new Error(`Error fetching ${url}`))
+  throw new Error(`Error fetching ${url}`)
 }
 
 function isRetryable (code: number): boolean {