Skip to content

refactor: remove @see test annotations in 4.8-only classes - #10625

Merged
paulbalandan merged 1 commit into
codeigniter4:4.8from
paulbalandan:remove-see-test-annotations-48
Oct 13, 2026
Merged

paulbalandan merged 1 commit into
codeigniter4:4.8from
paulbalandan:remove-see-test-annotations-48

Conversation

@paulbalandan

@paulbalandan paulbalandan commented Oct 12, 2026 •

Copy link
Copy Markdown
Member

Description
Complement to #10624 for the nine classes that exist only on 4.8: PlaceholderSampleGenerator, EnvironmentDetector, FormRequest, RequestInput, SSEResponse, StreamResponse, InputData, InputDataFactory and ValidatedInput. The @see \CodeIgniter\...Test annotation is removed from each class docblock.

The remaining 125 annotations on this branch are the ones shared with develop and are handled by that PR when it is merged down.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@carson-codeigniter4 carson-codeigniter4 Bot added 4.8 PRs that target the `4.8` branch. refactor Pull requests that refactor code labels Oct 12, 2026
@paulbalandan
paulbalandan force-pushed the remove-see-test-annotations-48 branch from 56d1881 to d09b079 Compare October 13, 2026 06:00
@paulbalandan paulbalandan changed the title refactor: replace @see test annotations in 4.8-only classes with #[CoversClass] in tests refactor: remove @see test annotations in 4.8-only classes Oct 13, 2026
@paulbalandan
paulbalandan merged commit b328325 into codeigniter4:4.8 Oct 13, 2026
61 checks passed
@paulbalandan
paulbalandan deleted the remove-see-test-annotations-48 branch October 13, 2026 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.8 PRs that target the `4.8` branch. refactor Pull requests that refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants