codeigniter4/CodeIgniter4

Dev: replace type `mixed` with more specific types

Fechada

#6.310 aberto em 29 de jul. de 2022

Ver no GitHubGuia do colaborador
 (11 comentários) (0 reação) (0 responsável)PHP (1.707 forks)batch import
devgood first issuehelp wanted

Métricas do repositório

Stars
 (4.492 estrelas)
Métricas de merge de PR
 (Mesclagem média 4d 15h) (68 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Pesquise todas as dicas de tipo `mixed` nos PHPDocs em toda a base de código e determine o tipo específico correto para cada uma com base no contexto (por exemplo, uso de parâmetros, valores de retorno).
Pilha de tecnologia
php
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PHPPHPDoc
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70