From 89191746ef74656b1dab9ee1608d31797912fd86 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=AD=99=E6=8C=AF=E5=AE=87?= <> Date: Wed, 22 Jan 2025 16:22:38 +0800 Subject: [PATCH] refactor(DependenciesResolver): replace echo with log.info for improved logging refactor(executeFreeleapsPipeline): update commit message linting condition for clarity MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: 孙振宇 <> --- .../src/com/freeleaps/devops/DependenciesResolver.groovy | 6 +++--- first-class-pipeline/vars/executeFreeleapsPipeline.groovy | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/first-class-pipeline/src/com/freeleaps/devops/DependenciesResolver.groovy b/first-class-pipeline/src/com/freeleaps/devops/DependenciesResolver.groovy index feb05f37..74cf33a5 100644 --- a/first-class-pipeline/src/com/freeleaps/devops/DependenciesResolver.groovy +++ b/first-class-pipeline/src/com/freeleaps/devops/DependenciesResolver.groovy @@ -33,12 +33,12 @@ class DependenciesResolver { steps.error("Dependencies manager is not set") } - echo "Ready to resolve dependencies for ${language}..." + log.info "Ready to resolve dependencies for ${language}..." - echo "Using ${mgr.manager} to resolve dependencies..." + log.info "Using ${mgr.manager} to resolve dependencies..." if (cachingEnabled) { - echo "Dependencies caching is enabled" + log.info "Dependencies caching is enabled" } switch (mgr) { diff --git a/first-class-pipeline/vars/executeFreeleapsPipeline.groovy b/first-class-pipeline/vars/executeFreeleapsPipeline.groovy index 9917b031..1174dbfe 100644 --- a/first-class-pipeline/vars/executeFreeleapsPipeline.groovy +++ b/first-class-pipeline/vars/executeFreeleapsPipeline.groovy @@ -34,8 +34,8 @@ def call(body) { script { def enabled = configurations.commitMessageLintEnabled - if (enabled != null && enabled == "true") { - log.info "Commit Line has enabled" + if (enabled == null || !enabled) { + log.warn "Commit message linting is disabled" } } }