Skip to content

Commit

Permalink
Merge pull request #26 from dinisnunes1/fix/remove-post-step
Browse files Browse the repository at this point in the history
[Fix] Run only the saveCache() on post step if cache argument is true
  • Loading branch information
amyu authored Mar 7, 2023
2 parents 9180762 + a7211ff commit 434ba07
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 7 deletions.
6 changes: 5 additions & 1 deletion dist/cleanup/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

17 changes: 11 additions & 6 deletions src/cleanup-android.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,13 +14,18 @@ async function run(): Promise<void> {
const buildToolsVersion = core.getInput(constants.INPUT_BUILD_TOOLS_VERSION)
const ndkVersion = core.getInput(constants.INPUT_NDK_VERSION)
const cmakeVersion = core.getInput(constants.INPUT_CMAKE_VERSION)
const cacheDisabled = core.getInput(constants.INPUT_CACHE_DISABLED)

let savedCacheEntry
if (!cacheDisabled) {
savedCacheEntry = await saveCache(
sdkVersion,
buildToolsVersion,
ndkVersion,
cmakeVersion
)
}

const savedCacheEntry = await saveCache(
sdkVersion,
buildToolsVersion,
ndkVersion,
cmakeVersion
)
await renderSummary(
sdkVersion,
buildToolsVersion,
Expand Down

0 comments on commit 434ba07

Please sign in to comment.