Add Rector for automated code modernisation and refactoring
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 38/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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?
- Langage dominant
- PHP
- Étoiles
- 20
- Forks
- 3
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de openapi/openapi-php-sdk
-
Improve PHPDoc Consistency Across the CodebasePeut-être à nouveau libre @Seraphim200001 l’a pris il y a 139 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
openapi/openapi-php-sdk#36 · 3 commentaires ·
-
enhancement help wanted
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
openapi/openapi-php-sdk#34 ·
-
good first issue question
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
openapi/openapi-php-sdk#33 · 2 commentaires ·
-
enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
openapi/openapi-php-sdk#32 ·
-
help wanted question
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
openapi/openapi-php-sdk#31 ·
Toutes les issues de openapi/openapi-php-sdk
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Automattic/blocks-engine#2803 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
hestiacp/hestiacp#5798 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
woocommerce/woocommerce-google-analytics-integration#699 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
wp-media/imagify-plugin#1289 ·
Les mainteneurs répondent en général sous 5 jours
-
Sidebar highlights two items when plugin routes share a prefix (/plugin/foo and /plugin/foo-bar)Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
getgrav/grav-plugin-admin2#195 ·