Add Rector for automated code modernisation and refactoring
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 38/100
Hướng nghiên cứu
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.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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?
- Ngôn ngữ chính
- PHP
- Star
- 20
- Fork
- 3
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của openapi/openapi-php-sdk
-
Improve PHPDoc Consistency Across the CodebaseCó thể làm lại được @Seraphim200001 đã nhận 139 ngày trước và không có pull request nào đang mở. Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
openapi/openapi-php-sdk#36 · 3 bình luận ·
-
enhancement help wanted
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
openapi/openapi-php-sdk#34 ·
-
good first issue question
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
openapi/openapi-php-sdk#33 · 2 bình luận ·
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
openapi/openapi-php-sdk#32 ·
-
help wanted question
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
openapi/openapi-php-sdk#31 ·
Tất cả issue của openapi/openapi-php-sdk
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Automattic/blocks-engine#2803 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
hestiacp/hestiacp#5798 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
woocommerce/woocommerce-google-analytics-integration#699 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
wp-media/imagify-plugin#1289 ·
Maintainer thường phản hồi trong vòng 5 ngày
-
Sidebar highlights two items when plugin routes share a prefix (/plugin/foo and /plugin/foo-bar)Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
getgrav/grav-plugin-admin2#195 ·