Merge pull request #3 from buildkite-plugins/bm/bash_compat

🤡 Ensure it works on old Bash

Author: mcncl

README.md

@@ -41,7 +41,7 @@ steps:
     command: |
       bazel build //... --build_event_json_file=bazel-events.json
     plugins:
-      - bazel-annotate#v0.1.0:
+      - bazel-annotate#v0.1.1:
           bep_file: bazel-events.json
 ```
 
@@ -55,7 +55,7 @@ steps:
       # Command might not produce a BEP file
       bazel build //...
     plugins:
-      - bazel-annotate#v0.1.0:
+      - bazel-annotate#v0.1.1:
           skip_if_no_bep: true
 ```
 
@@ -69,7 +69,7 @@ steps:
     command: |
       bazel test //... --build_event_json_file=bazel-test-events.json
     plugins:
-      - bazel-annotate#v0.1.0:
+      - bazel-annotate#v0.1.1:
           bep_file: bazel-test-events.json
 ```
 
@@ -82,7 +82,7 @@ steps:
       cd my-workspace
       bazel build //... --build_event_json_file=bazel-events.json
     plugins:
-      - bazel-annotate#v0.1.0:
+      - bazel-annotate#v0.1.1:
           bep_file: my-workspace/bazel-events.json
 ```
 
@@ -94,21 +94,21 @@ steps:
     command: |
       bazel build //... --build_event_json_file=bazel-build-events.json
     plugins:
-      - bazel-annotate#v0.1.0:
+      - bazel-annotate#v0.1.1:
           bep_file: bazel-build-events.json
 
   - label: "🧪 Test with Bazel"
     command: |
       bazel test //... --build_event_json_file=bazel-test-events.json
     plugins:
-      - bazel-annotate#v0.1.0:
+      - bazel-annotate#v0.1.1:
           bep_file: bazel-test-events.json
 
   - label: "📦 Package with Bazel"
     command: |
       bazel run //:package --build_event_json_file=bazel-package-events.json
     plugins:
-      - bazel-annotate#v0.1.0:
+      - bazel-annotate#v0.1.1:
           bep_file: bazel-package-events.json
 ```
 

lib/bazel-bep.bash

@@ -83,6 +83,7 @@ process_bep() {
 
   # Arrays for successful targets
   declare -a successful_targets=()
+  declare -a successful_targets_sorted=()
 
   # Arrays for test performance tracking
   declare -a slowest_tests=()
@@ -364,8 +365,9 @@ process_bep() {
 
 "
 
-    # Sort the targets for better readability
-    readarray -t successful_targets_sorted < <(printf '%s\n' "${successful_targets[@]}" | sort)
+if [ ${#successful_targets[@]} -gt 0 ]; then
+  mapfile -t successful_targets_sorted < <(printf '%s\n' "${successful_targets[@]}" | sort)
+fi
 
     # Show all targets
     for target in "${successful_targets_sorted[@]}"; do