Taking out commit comment to see if it causing charts not to get data

darrenge · Oct 15, 2024 · 5170ce6 · 5170ce6
1 parent e69c9c0
commit 5170ce6
Showing 1 changed file with 14 additions and 13 deletions.
diff --git a/.github/workflows/ci-bdnbenchmark.yml b/.github/workflows/ci-bdnbenchmark.yml
@@ -89,17 +89,18 @@ jobs:
           summary-always: true
           auto-push: true
 
+
       # Run `github-action-benchmark` action
-      - name: Store benchmark result for commit comment
-        uses: benchmark-action/github-action-benchmark@v1
-        with:
-          tool: 'benchmarkdotnet'
-          output-file-path: ./test/BDNPerfTests/BenchmarkDotNet.Artifacts/results/BDN.benchmark.${{ matrix.test }}-report-full-compressed.json
-          github-token: ${{ secrets.GITHUB_TOKEN }}
-          alert-threshold: '120%' 
-          external-data-json-path: ./cache/benchmark-data.json
-          comment-on-alert: true
-          fail-on-alert: true
-          alert-comment-cc-users: '@darrenge'
-          summary-always: false
-          auto-push: false  # needs to be false if using external-data-json-path
+    #  - name: Store benchmark result for commit comment
+#        uses: benchmark-action/github-action-benchmark@v1
+#        with:
+#          tool: 'benchmarkdotnet'
+#          output-file-path: ./test/BDNPerfTests/BenchmarkDotNet.Artifacts/results/BDN.benchmark.${{ matrix.test }}-report-full-compressed.json
+#          github-token: ${{ secrets.GITHUB_TOKEN }}
+#          alert-threshold: '120%' 
+#          external-data-json-path: ./cache/benchmark-data.json
+#          comment-on-alert: true
+#          fail-on-alert: true
+#          alert-comment-cc-users: '@darrenge'
+#          summary-always: false
+#          auto-push: false  # needs to be false if using external-data-json-path