Add Rector for automated code modernisation and refactoring
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 38/100
Direção de pesquisa
Start by reading composer.json and the constructor in src/Client.php, then check Rector’s PHP 8.1 rules against the SDK’s PHP 8.0 support. Decide which rule sets are safe and whether CI should run Rector; done means the dependency, configuration, and dry-run and fix scripts are defined with that compatibility decision documented.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Motivation
As PHP evolves and the SDK targets PHP 8.0+, there are patterns that could be automatically modernised: constructor promotion, match expressions, named arguments, and more.
Example
Current code in src/Client.php:
public function __construct(?string $token = null, ...)
{
$this->token = $token ?? getenv('OPEN_API_TOKEN');
...
}
With Rector + constructor promotion this could become:
public function __construct(
private ?string $token = null,
...
) {
$this->token ??= getenv('OPEN_API_TOKEN');
}
Proposal
- Add
rector/rectortorequire-dev - Configure a
rector.phpwith theLevelSetList::UP_TO_PHP_81preset - Add a
composer run rectorscript for dry-run andcomposer run rector-fixto apply
Open questions
- Should Rector run in CI as a check-only step, or only locally?
- Which rule sets are safe to apply without risking breaking changes for consumers on PHP 8.0?
- Linguagem predominante
- PHP
- Estrelas
- 20
- Forks
- 3
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de openapi/openapi-php-sdk
-
Improve PHPDoc Consistency Across the CodebaseTalvez livre de novo @Seraphim200001 assumiu há 139 dias e não há nenhum pull request aberto. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 48/100
openapi/openapi-php-sdk#36 · 3 comentários ·
-
enhancement help wanted
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
openapi/openapi-php-sdk#34 ·
-
good first issue question
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
openapi/openapi-php-sdk#33 · 2 comentários ·
-
enhancement
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
openapi/openapi-php-sdk#32 ·
-
help wanted question
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
openapi/openapi-php-sdk#31 ·
Todas as issues de openapi/openapi-php-sdk
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
matomo-org/device-detector#8405 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
Mantenedores costumam responder em até 1 dia
-
sync-en
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
Mantenedores costumam responder em até 1 dia
-
sync-en
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
Mantenedores costumam responder em até 4 dias