Skip to content

Commit

Permalink
fix: ci: avoid failing the CI if the bench result is not present
Browse files Browse the repository at this point in the history
This is mostly because when we are adding a new benchmark in a PR; the benchmark won't be yet available in main, therefore the CI will fail.
  • Loading branch information
dsocolobsky authored and lima-limon-inc committed Jan 29, 2025
1 parent ed71032 commit 8595dd8
Showing 1 changed file with 14 additions and 14 deletions.
28 changes: 14 additions & 14 deletions .github/workflows/ci_bench_levm_in_pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,19 +38,19 @@ jobs:
make revm-comparison-ci
{
echo "#### Benchmark Results: Factorial";
cat Factorial.md;
cat Factorial.md || echo "No results";
echo "#### Benchmark Results: Factorial - Recursive";
cat FactorialRecursive.md;
cat FactorialRecursive.md || echo "No results";
echo "#### Benchmark Results: Fibonacci";
cat Fibonacci.md;
cat Fibonacci.md || echo "No results";
echo "#### Benchmark Results: ManyHashes";
cat ManyHashes.md;
cat ManyHashes.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Transfer";
cat ERC20Transfer.md;
cat ERC20Transfer.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Mint";
cat ERC20Mint.md;
cat ERC20Mint.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Approval";
cat ERC20Approval.md;
cat ERC20Approval.md || echo "No results";
} > pr_result.md
- name: Upload PR results
Expand Down Expand Up @@ -90,19 +90,19 @@ jobs:
make revm-comparison-ci
{
echo "#### Benchmark Results: Factorial";
cat Factorial.md;
cat Factorial.md || echo "No results";
echo "#### Benchmark Results: Factorial - Recursive";
cat FactorialRecursive.md;
cat FactorialRecursive.md || echo "No results";
echo "#### Benchmark Results: Fibonacci";
cat Fibonacci.md;
cat Fibonacci.md || echo "No results";
echo "#### Benchmark Results: ManyHashes";
cat ManyHashes.md;
cat ManyHashes.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Transfer";
cat ERC20Transfer.md;
cat ERC20Transfer.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Mint";
cat ERC20Mint.md;
cat ERC20Mint.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Approval";
cat ERC20Approval.md;
cat ERC20Approval.md || echo "No results";
} > main_result.md
- name: Upload main results
Expand Down

0 comments on commit 8595dd8

Please sign in to comment.