(beam) 01/01: Use onlyif

[vterentev]

This is an automated email from the ASF dual-hosted git repository.

vterentev pushed a commit to branch fix-precommit-website
in repository https://gitbox.apache.org/repos/asf/beam.git

commit 7b46f792ae7e6678a6516c3119b0283193644b43
Author: Vitaly Terentyev <vitaly.terent...@akvelon.com>
AuthorDate: Thu Feb 12 09:57:48 2026 +0400

    Use onlyif
---
 website/build.gradle | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/website/build.gradle b/website/build.gradle
index 790afe582eb..2aa04282e67 100644
--- a/website/build.gradle
+++ b/website/build.gradle
@@ -135,10 +135,13 @@ task copyGcpCredsToContainer(type: Exec) {
   dependsOn setupDockerContainer
 
   def gcpCredsHost = providers.gradleProperty("gcpCredsFile").orNull
-  if (gcpCredsHost) {
-    commandLine 'docker', 'cp', gcpCredsHost, 
"${->startDockerContainer.containerId()}:${gcpCredsInContainer}"
-  } else {
-    println "[performance_looks] -PgcpCredsFile not provided, skipping copy"
+
+  onlyIf { gcpCredsHost != null }
+
+  doFirst {
+    commandLine 'docker', 'cp',
+      gcpCredsHost,
+      "${->startDockerContainer.containerId()}:${gcpCredsInContainer}"
   }
 }
 
@@ -146,7 +149,10 @@ task downloadPerformanceLooks(type: Exec) {
   dependsOn setupDockerContainer, copyGcpCredsToContainer
 
   def gcpCredsHost = providers.gradleProperty("gcpCredsFile").orNull
-  if (gcpCredsHost) {
+
+  onlyIf { gcpCredsHost != null }
+
+  doFirst {
     commandLine 'docker', 'exec',
       "${-> setupDockerContainer.containerId()}", '/bin/bash', '-lc',
       """
@@ -173,8 +179,6 @@ task downloadPerformanceLooks(type: Exec) {
         echo "[performance_looks][WARNING] Website build will continue without 
updated images"
       fi
     """
-  } else {
-    println "[performance_looks] Skipping download of performance looks"
   }
 }