Add Rector for automated code modernisation and refactoring
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 38/100
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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?
- Lingua principale
- PHP
- Stelle
- 20
- Fork
- 3
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di openapi/openapi-php-sdk
-
Improve PHPDoc Consistency Across the CodebaseForse di nuovo libera @Seraphim200001 l’ha presa 140 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
openapi/openapi-php-sdk#36 · 3 commenti ·
-
enhancement help wanted
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
openapi/openapi-php-sdk#34 ·
-
good first issue question
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
openapi/openapi-php-sdk#33 · 2 commenti ·
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
openapi/openapi-php-sdk#32 ·
-
help wanted question
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
openapi/openapi-php-sdk#31 ·
Tutte le issue di openapi/openapi-php-sdk
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
Automattic/blocks-engine#2866 ·
I maintainer di solito rispondono entro 1 giorno
-
Good first issue: add laravel-inertia stack budget preset (Laravel 11 + Inertia.js + PHP 8.3)Apertaenhancement good first issue help wanted
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
backend gamification P0
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
semanadeinformatica/website#461 ·
-
HttpRequestTrait fatals on WordPress 7.2: $blocked_http_requests clashes with WP_UnitTestCase_BaseAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
area: docs eco: php good first issue help wanted lane: free-agent roadmap size: XS type: docs
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
arminazimi/evergreen#453 ·
I maintainer di solito rispondono entro 1 giorno