Skip to content

Report processed annotation counts - #129

Open
oiahoon wants to merge 1 commit into
staabm:masterfrom
oiahoon:fix/report-processed-counts
Open

oiahoon wants to merge 1 commit into
staabm:masterfrom
oiahoon:fix/report-processed-counts

Conversation

@oiahoon

@oiahoon oiahoon commented Jul 11, 2026

Copy link
Copy Markdown

Summary

  • count emitted error and warning annotations
  • report the final counts to STDERR after processing completes
  • keep GitHub annotation output on STDOUT unchanged
  • update the test harness to assert STDOUT and STDERR independently

Why

An empty STDOUT could mean either that a report contained no errors or warnings, or that processing produced no visible result. A final STDERR status makes successful processing explicit without changing the annotation stream consumed by GitHub Actions.

Closes #28.

Validation

  • all 19 existing report and option scenarios pass
  • mixed report keeps the same three STDOUT annotations
  • mixed report writes Processed errors: 2; warnings: 1. to STDERR
  • empty reports write zero error and warning counts
  • malformed XML retains its existing STDERR diagnostics
  • PHP syntax checks for cs2pr and tests/tests.php
  • Composer manifest validation
  • repository PHP CS Fixer dry-run
  • Git diff check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

emit status report

1 participant