Fix aggregator Cardano transaction prover benchmark tool #1790

jpraynaud · 2024-06-28T14:21:55Z

Content

This PR includes fix to the aggregator Cardano transaction prover benchmark tool which was creating invalid requests.

Branch
- Commit sequence broadly makes sense
- Key commits have useful messages
PR
- No clippy warnings in the CI
- Self-reviewed the diff
- Useful pull request description
- Reviewer requested

Relates to #1757

github-actions · 2024-06-28T14:28:24Z

4 files ±0 51 suites ±0 8m 42s ⏱️ -5s
1 101 tests ±0 1 101 ✅ ±0 0 💤 ±0 0 ❌ ±0
1 249 runs ±0 1 249 ✅ ±0 0 💤 ±0 0 ❌ ±0

Results for commit 5791811. ± Comparison against base commit 84f0f39.

jpraynaud added 2 commits June 28, 2024 16:18

fix: transaction mainnet test list sanitized

4e95962

fix: remove trailing comma in prover url computed by benchmark script

5791811

jpraynaud self-assigned this Jun 28, 2024

jpraynaud requested review from Alenar, sfauvel and dlachaume June 28, 2024 14:22

jpraynaud temporarily deployed to testing-preview June 28, 2024 14:28 — with GitHub Actions Inactive

jpraynaud temporarily deployed to testing-sanchonet June 28, 2024 14:28 — with GitHub Actions Inactive

Alenar approved these changes Jun 28, 2024

View reviewed changes

jpraynaud merged commit ed03ede into main Jul 1, 2024
43 checks passed

jpraynaud deleted the jpraynaud/1757-fix-benchmark-aggregator-prover branch July 1, 2024 09:13