Skip to content

refactor: fix minor type inaccuracies found via PHPStan bleeding edge - #10522

Merged
paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:bleeding-edge-cheap-wins
Sep 5, 2026
Merged

refactor: fix minor type inaccuracies found via PHPStan bleeding edge#10522
paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:bleeding-edge-cheap-wins

Conversation

@paulbalandan

Copy link
Copy Markdown
Member

Description
I'm trying to enable bleeding edge here but the 188 errors are not flexible enough to be done in one pass. Landing the cheap wins first.

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

@paulbalandan paulbalandan added the refactor Pull requests that refactor code label Sep 4, 2026
@michalsn

michalsn commented Sep 4, 2026

Copy link
Copy Markdown
Member

Just to be sure: you're not planning to eventually switch develop to bleeding-edge PHPStan here, right?

Comment thread system/Database/Database.php
@paulbalandan
paulbalandan force-pushed the bleeding-edge-cheap-wins branch from 797bf69 to e430f85 Compare September 5, 2026 11:20
@paulbalandan

Copy link
Copy Markdown
Member Author

Just to be sure: you're not planning to eventually switch develop to bleeding-edge PHPStan here, right?

Yes, not planning to have bleeding edge enabled. Just fixing any errors from it.

@paulbalandan
paulbalandan force-pushed the bleeding-edge-cheap-wins branch from e430f85 to d6ad142 Compare September 5, 2026 12:03
@paulbalandan
paulbalandan merged commit 860b5ec into codeigniter4:develop Sep 5, 2026
55 checks passed
@paulbalandan
paulbalandan deleted the bleeding-edge-cheap-wins branch September 5, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Pull requests that refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants