Skip to content

Commit 9ff8af0

Browse files
committed
fix(ci): use phpbench dump-file output for v1.6 compatibility
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
1 parent 7ae3d11 commit 9ff8af0

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

.github/copilot-instructions.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ vendor-bin/phpbench/vendor/phpbench/phpbench/bin/phpbench run --bootstrap=vendor
2727
PHPBench automatically:
2828
- Runs warmup iterations (eliminates JIT/opcache startup noise)
2929
- Executes multiple revisions for statistical confidence
30-
- Reports mean, min, max, stdev, variance per benchmark
30+
- Reports mean, min, max, stdev, variance per benchmark (CI dump file: `build/benchmark-results.xml`)
3131

3232
## Compliance and contribution
3333

.github/workflows/performance.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,13 @@ jobs:
3434
3535
echo "✓ Benchmarks completed"
3636
echo ""
37-
echo "Results saved to build/benchmark-results.json"
37+
echo "Results saved to build/benchmark-results.xml"
3838
echo "Review: Ensure no individual benchmark takes >10ms average"
3939
4040
- name: Save benchmark results as artifact
4141
if: always()
4242
uses: actions/upload-artifact@v4
4343
with:
4444
name: benchmark-results
45-
path: build/benchmark-results.json
45+
path: build/benchmark-results.xml
4646
retention-days: 90

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@
7676
"test:coverage": "XDEBUG_MODE=coverage vendor-bin/phpunit/vendor/phpunit/phpunit/phpunit --coverage-text --coverage-clover=build/coverage/clover.xml",
7777
"mutation:test": "vendor-bin/mutation/vendor/infection/infection/bin/infection --threads=max",
7878
"benchmark:run": "vendor-bin/phpbench/vendor/phpbench/phpbench/bin/phpbench run --bootstrap=vendor/autoload.php --report=aggregate benchmarks",
79-
"benchmark:run:ci": "mkdir -p build && vendor-bin/phpbench/vendor/phpbench/phpbench/bin/phpbench run --bootstrap=vendor/autoload.php --iterations=20 --revs=10 --warmup=2 --report=aggregate --output=json:build/benchmark-results.json benchmarks",
79+
"benchmark:run:ci": "mkdir -p build && vendor-bin/phpbench/vendor/phpbench/phpbench/bin/phpbench run --bootstrap=vendor/autoload.php --iterations=20 --revs=10 --warmup=2 --report=aggregate --dump-file=build/benchmark-results.xml benchmarks",
8080
"bin": [
8181
"composer bin all list"
8282
]

0 commit comments

Comments
 (0)