Skip to content

Final test coverage sweep - #42

Merged
jordikroon merged 24 commits into
php:mainfrom
NickSdot:stack-11-coverage
Jul 26, 2026
Merged

Final test coverage sweep#42
jordikroon merged 24 commits into
php:mainfrom
NickSdot:stack-11-coverage

Conversation

@NickSdot

Copy link
Copy Markdown
Contributor

Mostly Codex-identified gaps and edge cases; made sure that the vast majority is not academic crap and actual value is added; some ignores weren't avoidable.

Depends on #29, #30, #31, #32, #33, #34, #35, #36, #37, #38, #39, #40 and #41.
This PRs diff will shrink as preceding PRs are merged and this branch is rebased.

Diff against previous branch: https://github.com/php/docbook-cs/compare/NickSdot:stack-10-reporting..NickSdot:stack-11-coverage (different to diff in here only until the previous branch was merged)

@NickSdot
NickSdotforce-pushed the stack-11-coverage branch 5 times, most recently from b37f80c to a1b7011CompareJuly 25, 2026 17:38
@NickSdot
NickSdot marked this pull request as ready for review July 25, 2026 17:50
@NickSdot

Copy link
Copy Markdown
ContributorAuthor

Rebased.

Comment threadsrc/Sniff/SimparaSniff.php Outdated
Comment threadsrc/Report/Report.php
@jordikroon
jordikroon merged commit 865fb5d into php:mainJul 26, 2026
3 checks passed
Sign up for freeto 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.

2 participants

@NickSdot@jordikroon