CI: Parametrize scripts for module/build variant (diff merged manifest)

woocommerce · Feb 21, 2025 · a2993ee · a2993ee
1 parent 15fed89
commit a2993ee
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 17 deletions.
diff --git a/.buildkite/commands/diff-merged-manifest.sh b/.buildkite/commands/diff-merged-manifest.sh
@@ -1,8 +1,10 @@
 #!/bin/bash
 
-"$(dirname "${BASH_SOURCE[0]}")/restore-merged-manifest.sh" "WooCommerce" "jalapenoDebug"
+BUILD_VARIANT=$1
 
-"$(dirname "${BASH_SOURCE[0]}")/restore-merged-manifest.sh" "WooCommerce-Wear" "jalapenoDebug"
+"$(dirname "${BASH_SOURCE[0]}")/restore-merged-manifest.sh" "WooCommerce" ${BUILD_VARIANT}
+
+"$(dirname "${BASH_SOURCE[0]}")/restore-merged-manifest.sh" "WooCommerce-Wear" ${BUILD_VARIANT}
 
 "$(dirname "${BASH_SOURCE[0]}")/restore-cache.sh"
 
@@ -14,12 +16,12 @@ install_gems
 echo "--- :closed_lock_with_key: Installing Secrets"
 bundle exec fastlane run configure_apply
 
-echo "--- 📦 Create Merged Manifest"
-./gradlew assembleJalapenoDebug
+echo "--- 📦 Create Merged Manifest (Build Variant: ${BUILD_VARIANT})"
+./gradlew assemble"${BUILD_VARIANT^}"
 echo ""
 
-echo "--- 💾 Diff Merged Manifest for WooCommerce"
-comment_with_manifest_diff "WooCommerce" "jalapenoDebug"
+echo "--- 💾 Diff Merged Manifest (Module: WooCommerce, Build Variant: ${BUILD_VARIANT})"
+comment_with_manifest_diff "WooCommerce" ${BUILD_VARIANT}
 
-echo "--- 💾 Diff Merged Manifest for WooCommerce-Wear"
-comment_with_manifest_diff "WooCommerce-Wear" "jalapenoDebug"
+echo "--- 💾 Diff Merged Manifest (Module: WooCommerce-Wear, Build Variant: ${BUILD_VARIANT})"
+comment_with_manifest_diff "WooCommerce-Wear" ${BUILD_VARIANT}
diff --git a/.buildkite/commands/restore-merged-manifest.sh b/.buildkite/commands/restore-merged-manifest.sh
@@ -3,5 +3,5 @@
 MODULE=$1
 BUILD_VARIANT=$2
 
-echo "--- 💾 Restore Debug Manifest (Jalapeno)"
+echo "--- 💾 Restore Merged Manifest (Module: ${MODULE}, Build Variant: ${BUILD_VARIANT})"
 restore_android_merged_manifest ${MODULE} ${BUILD_VARIANT} || true
diff --git a/.buildkite/commands/save-merged-manifest.sh b/.buildkite/commands/save-merged-manifest.sh
@@ -1,5 +1,7 @@
 #!/bin/bash
 
+BUILD_VARIANT=$1
+
 "$(dirname "${BASH_SOURCE[0]}")/restore-cache.sh"
 
 set -euo pipefail
@@ -10,12 +12,12 @@ install_gems
 echo "--- :closed_lock_with_key: Installing Secrets"
 bundle exec fastlane run configure_apply
 
-echo "--- 📦 Create Merged Manifest"
-./gradlew assembleJalapenoDebug
+echo "--- 📦 Create Merged Manifest (Build Variant: ${BUILD_VARIANT})"
+./gradlew assemble"${BUILD_VARIANT^}"
 echo ""
 
-echo "--- 💾 Save Merged Manifest for WooCommerce"
-save_android_merged_manifest "WooCommerce" "jalapenoDebug"
+echo "--- 💾 Save Merged Manifest (Module: WooCommerce, Build Variant: ${BUILD_VARIANT})"
+save_android_merged_manifest "WooCommerce" ${BUILD_VARIANT}
 
-echo "--- 💾 Save Merged Manifest for WooCommerce-Wear"
-save_android_merged_manifest "WooCommerce-Wear" "jalapenoDebug"
+echo "--- 💾 Save Merged Manifest (Module: WooCommerce-Wear, Build Variant: ${BUILD_VARIANT})"
+save_android_merged_manifest "WooCommerce-Wear" ${BUILD_VARIANT}
diff --git a/.buildkite/pipeline.yml b/.buildkite/pipeline.yml
@@ -6,7 +6,7 @@ agents:
 
 steps:
   - label: "💾 Restore and Diff Merged Manifest"
-    command: .buildkite/commands/diff-merged-manifest.sh
+    command: ".buildkite/commands/diff-merged-manifest.sh jalapenoDebug"
     plugins: [$CI_TOOLKIT]
     artifact_paths:
       - "**/build/reports/diff_manifest/*"
diff --git a/.buildkite/schedules/merged-manifest-storage.yml b/.buildkite/schedules/merged-manifest-storage.yml
@@ -6,5 +6,5 @@ agents:
 
 steps:
   - label: "💾 Create and Store Merged Manifest"
-    command: .buildkite/commands/save-merged-manifest.sh
+    command: ".buildkite/commands/save-merged-manifest.sh jalapenoDebug"
     plugins: [$CI_TOOLKIT]