Merge pull request #1160 from gregdhill/fix/escape-bench-syntax

fix: leave bench args unset
interlay · Aug 3, 2023 · 32e1f20 · 32e1f20
2 parents 0b5a0ce + a34e625
commit 32e1f20
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/.github/workflows/benchmark.yml b/.github/workflows/benchmark.yml
@@ -41,7 +41,7 @@ jobs:
           script: |
             const [cmd, ...args] = context.payload.comment.body.split(" ")
             const [runtime, pallet] = args
-            return `bash ./scripts/benchmark.sh -r ${runtime ?? "\\''*'\\'"} -p ${pallet ?? "\\''*'\\'"}`
+            return `bash ./scripts/benchmark.sh -r ${runtime ?? ""} -p ${pallet ?? ""}`
       - uses: actions/github-script@v6
         name: Post comment
         id: comment

diff --git a/scripts/benchmark.sh b/scripts/benchmark.sh
@@ -21,6 +21,10 @@ while getopts ":r:p:" opt; do
   esac
 done
 
+if [ -z "${pallet}" ]; then
+  pallet='*'
+fi
+
 cargo run \
   --bin interbtc-parachain \
   --features runtime-benchmarks \