codeigniter4/CodeIgniter4

Dev: replace type `mixed` with more specific types

Open

#6,310 opened on 2022年7月29日

GitHub で見る
 (11 comments) (0 reactions) (0 assignees)PHP (4,492 stars) (1,707 forks)batch import
devgood first issuehelp wanted

説明

Type mixed means object|resource|array|string|int|float|bool|null. https://www.php.net/manual/en/language.types.declarations.php#language.types.declarations.mixed

The current code base has a lot of type mixed in PHPDocs, But I think all of them are not correct.

And mixed prevents PHPStan level from increasing.

PHPStan’s rule level 6 isn’t satisfied with implicit mixed https://phpstan.org/writing-php-code/phpdoc-types#mixed

We need to replace type mixed with more specific types.

コントリビューターガイド

技術スタック
なし
領域
backend
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
PHP basicsunderstanding of PHPDoc type declarationsfamiliarity with PHPStan
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70
調査方針
Search the codebase for occurrences of `mixed` in PHPDoc comments (e.g., using grep or an IDE). Replace each with the most specific type as per PHP documentation (e.g., `string`, `int`, `array`). Refer to PHPStan's rule level 6 expectations. No linked PRs or maintainer questions as of now.