From 3f36c213b482a305939e55e85ac6ff84157bd683 Mon Sep 17 00:00:00 2001 From: Fernandez Ludovic Date: Thu, 25 Apr 2024 02:44:27 +0200 Subject: [PATCH] feat: remove Go cache management --- README.md | 8 +------- action.yml | 8 -------- src/cache.ts | 24 ++---------------------- 3 files changed, 3 insertions(+), 37 deletions(-) diff --git a/README.md b/README.md index 14b77caa4c..fff461ebfa 100644 --- a/README.md +++ b/README.md @@ -49,7 +49,7 @@ jobs: # Require: The version of golangci-lint to use. # When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version. # When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit. - version: v1.54 + version: v1.57 # Optional: working directory, useful for monorepos # working-directory: somedir @@ -67,12 +67,6 @@ jobs: # takes precedence over all other caching options. # skip-cache: true - # Optional: if set to true, then the action won't cache or restore ~/go/pkg. - # skip-pkg-cache: true - - # Optional: if set to true, then the action won't cache or restore ~/.cache/go-build. - # skip-build-cache: true - # Optional: The mode to install golangci-lint. It can be 'binary' or 'goinstall'. # install-mode: "goinstall" ``` diff --git a/action.yml b/action.yml index 3b3ea21c27..23cb5a3902 100644 --- a/action.yml +++ b/action.yml @@ -29,14 +29,6 @@ inputs: takes precedence over all other caching options. default: 'false' required: false - skip-pkg-cache: - description: "if set to true then the action doesn't cache or restore ~/go/pkg." - default: 'false' - required: false - skip-build-cache: - description: "if set to true then the action doesn't cache or restore ~/.cache/go-build." - default: 'false' - required: false install-mode: description: "The mode to install golangci-lint. It can be 'binary' or 'goinstall'." default: "binary" diff --git a/src/cache.ts b/src/cache.ts index cb8f828ac9..d13fb6a7de 100644 --- a/src/cache.ts +++ b/src/cache.ts @@ -23,26 +23,6 @@ const getLintCacheDir = (): string => { return path.resolve(`${process.env.HOME}/.cache/golangci-lint`) } -const getCacheDirs = (): string[] => { - // Not existing dirs are ok here: it works. - const skipPkgCache = core.getInput(`skip-pkg-cache`, { required: true }).trim() - const skipBuildCache = core.getInput(`skip-build-cache`, { required: true }).trim() - const dirs = [getLintCacheDir()] - - if (skipBuildCache.toLowerCase() == "true") { - core.info(`Omitting ~/.cache/go-build from cache directories`) - } else { - dirs.push(path.resolve(`${process.env.HOME}/.cache/go-build`)) - } - if (skipPkgCache.toLowerCase() == "true") { - core.info(`Omitting ~/go/pkg from cache directories`) - } else { - dirs.push(path.resolve(`${process.env.HOME}/go/pkg`)) - } - - return dirs -} - const getIntervalKey = (invalidationIntervalDays: number): string => { const now = new Date() const secondsSinceEpoch = now.getTime() / 1000 @@ -97,7 +77,7 @@ export async function restoreCache(): Promise { } core.saveState(State.CachePrimaryKey, primaryKey) try { - const cacheKey = await cache.restoreCache(getCacheDirs(), primaryKey, restoreKeys) + const cacheKey = await cache.restoreCache([getLintCacheDir()], primaryKey, restoreKeys) if (!cacheKey) { core.info(`Cache not found for input keys: ${[primaryKey, ...restoreKeys].join(", ")}`) return @@ -127,7 +107,7 @@ export async function saveCache(): Promise { const startedAt = Date.now() - const cacheDirs = getCacheDirs() + const cacheDirs = [getLintCacheDir()] const primaryKey = core.getState(State.CachePrimaryKey) if (!primaryKey) { utils.logWarning(`Error retrieving key from state.`)