From 81b3095925df4482430723e2815b6a700f3dc689 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=AD=99=E6=8C=AF=E5=AE=87?= <> Date: Fri, 7 Feb 2025 17:14:11 +0800 Subject: [PATCH] fix(caching): remove redundant logging and streamline cache existence checks for npm and yarn installations MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: 孙振宇 <> --- .../devops/DependenciesResolver.groovy | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/first-class-pipeline/src/com/freeleaps/devops/DependenciesResolver.groovy b/first-class-pipeline/src/com/freeleaps/devops/DependenciesResolver.groovy index 3a6a914b..1f6c3b12 100644 --- a/first-class-pipeline/src/com/freeleaps/devops/DependenciesResolver.groovy +++ b/first-class-pipeline/src/com/freeleaps/devops/DependenciesResolver.groovy @@ -66,11 +66,10 @@ class DependenciesResolver { if (cachingEnabled) { steps.dir(this.workspace) { steps.cache(maxCacheSize: 512, caches: [[$class: 'ArbitraryFileCache', includes: '**/*', path: '.npm-cache', cacheValidityDecidingFile: 'package-lock.json']]) { - steps.log.info("Dependencies Resolver","Caches are restored") - } - def cacheExists = steps.fileExists('.npm-cache') - if (!cacheExists) { - steps.sh "npm install --cache .npm-cache" + def cacheExists = steps.fileExists('.npm-cache') + if (!cacheExists) { + steps.sh "npm install --cache .npm-cache" + } } } } else { @@ -92,11 +91,10 @@ class DependenciesResolver { if (cachingEnabled) { steps.dir(this.workspace) { steps.cache(maxCacheSize: 512, caches: [[$class: 'ArbitraryFileCache', includes: '**/*', path: '.yarn-cache', cacheValidityDecidingFile: 'yarn.lock']]) { - steps.log.info("Dependencies Resolver","Caches are restored") - } - def cacheExists = steps.fileExists('.yarn-cache') - if (!cacheExists) { - steps.sh "yarn install --cache-folder .yarn-cache" + def cacheExists = steps.fileExists('.yarn-cache') + if (!cacheExists) { + steps.sh "yarn install --cache-folder .yarn-cache" + } } } } else {