Skip to content

Add CI/coverage badges and integration examples - #3

Merged
mohaelmrabet merged 1 commit into
mainfrom
docs/coverage-and-examples
Sep 4, 2026
Merged

mohaelmrabet merged 1 commit into
mainfrom
docs/coverage-and-examples

Conversation

@mohaelmrabet

Copy link
Copy Markdown
Contributor

CI: new coverage job (pcov + Codecov upload, fail_ci_if_error: false so CI never blocks on it before the Codecov app is enabled for this repo — a manual activation step on codecov.io, same category as the Packagist submission). README: Tests and codecov badges.

examples/: basic.php (a PHP fatal that is not a Throwable, caught via the shutdown handler), psr3-logger.php (a PSR-3 logger writing to stdout, 12-factor style), sse-interception.php (a fatal mid-stream on an active SSE response, answered as one more SSE event instead of corrupting the stream), slim.php (where this boundary sits relative to Slim's own error middleware — installed before the app, answers what Slim's middleware structurally cannot: a non-Throwable fatal).

Each snippet other than slim.php was run standalone and its output verified. None add a runtime dependency to the package itself.

CI: new 'coverage' job (pcov + Codecov upload, fail_ci_if_error: false so
CI never blocks on it before the Codecov app is enabled for this repo).
README: Tests and codecov badges.

examples/: basic.php (a PHP fatal that is not a Throwable, caught via the
shutdown handler), psr3-logger.php (a PSR-3 logger writing to stdout,
12-factor style), sse-interception.php (a fatal mid-stream on an active
SSE response, answered as one more SSE event instead of corrupting the
stream), slim.php (where this boundary sits relative to Slim's own error
middleware). Each snippet other than slim.php was run and its output
verified; none add a dependency to the package itself.
@mohaelmrabet
mohaelmrabet merged commit 9128150 into main Sep 4, 2026
9 checks passed
@mohaelmrabet
mohaelmrabet deleted the docs/coverage-and-examples branch September 4, 2026 21:48
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.

1 participant