ci: fix encoding benchmarks only on changes (#133)

Author: abemedia

.github/workflows/bechmark.yml

@@ -28,11 +28,15 @@ jobs:
         run: GO111MODULE=off go get golang.org/x/perf/cmd/benchstat
 
       - name: Enable encoding benchmarks on changes
-        run: |
-          echo "BENCHMARK_ENCODING=$(git diff master HEAD --name-only -- encoding/)" >> $GITHUB_ENV
+        id: encoding-changed
+        uses: tj-actions/changed-files@v35
+        with:
+          files: encoding/**
 
       - name: Run benchmark
         run: go test -bench=. -count=10 -benchmem ./... | tee bench-new.txt
+        env:
+          BENCHMARK_ENCODING: ${{ steps.encoding-changed.outputs.any_changed }}
 
       - name: Run benchmark for base code
         run: |
@@ -42,6 +46,8 @@ jobs:
           go test -bench=. -count=10 -benchmem ./... | tee bench-master.txt
           benchstat bench-master.txt
           git reset --hard $HEAD
+        env:
+          BENCHMARK_ENCODING: ${{ steps.encoding-changed.outputs.any_changed }}
 
       - name: Benchmark stats
         id: bench

internal/test/encoding.go

@@ -6,6 +6,7 @@ import (
 	"net/http"
 	"os"
 	"reflect"
+	"strconv"
 	"strings"
 	"testing"
 
@@ -99,7 +100,7 @@ func BenchmarkEncoding[T any](b *testing.B, opt EncodingOptions[T]) {
 }
 
 func BenchmarkDecode[T any](b *testing.B, opt EncodingOptions[T]) {
-	if os.Getenv("BENCHMARK_ENCODING") == "" {
+	if ok, _ := strconv.ParseBool(os.Getenv("BENCHMARK_ENCODING")); !ok {
 		b.SkipNow()
 	}
 
@@ -119,7 +120,7 @@ func BenchmarkDecode[T any](b *testing.B, opt EncodingOptions[T]) {
 }
 
 func BenchmarkEncode[T any](b *testing.B, opt EncodingOptions[T]) {
-	if os.Getenv("BENCHMARK_ENCODING") == "" {
+	if ok, _ := strconv.ParseBool(os.Getenv("BENCHMARK_ENCODING")); !ok {
 		b.SkipNow()
 	}