Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Rector] Update to rector 0.12.15, apply parallel, and RemoveUnusedPrivatePropertyRector #489

Merged
merged 4 commits into from
Jan 26, 2022

Conversation

samsonasik
Copy link
Contributor

@samsonasik samsonasik commented Jan 22, 2022

Q A
QA? ✔️
  • updated rector dependency to 0.12.15
  • apply parallel process to speed up check
  • apply RemoveUnusedPrivatePropertyRector
  • Enable allow-plugins for composer 2.2 support.

@samsonasik
Copy link
Contributor Author

All green 🎉

@wolfy-j wolfy-j requested a review from butschster January 22, 2022 11:57
rector.php Outdated Show resolved Hide resolved
@samsonasik samsonasik changed the title [Rector] Update to rector 0.12.13, apply parallel, and RemoveUnusedPrivatePropertyRector [Rector] Update to rector 0.12.14, apply parallel, and RemoveUnusedPrivatePropertyRector Jan 26, 2022
@samsonasik
Copy link
Contributor Author

samsonasik commented Jan 26, 2022

updated to use rector 0.12.14 and re-run rector so option skip for RemoveUnusedPrivatePropertyRector now cleaned up.

@@ -36,7 +33,6 @@ public function __construct(
string $footer = ''
) {
$this->function = $function;
$this->parameters = $parameters;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only remove property and its assignment in constructor for unused private property without removing its middle __construct parameter that fill it to avoid BC break.

@samsonasik
Copy link
Contributor Author

All green 🎉

@butschster
Copy link
Member

Hi @samsonasik! Thank you very much! You are the best! We are going to start working on spiral framework 3.0 and it would be great to prepare rector for php 8.0!

@samsonasik
Copy link
Contributor Author

@butschster yes 👍 , just let me know there is a branch 3.0 to give php 8.0 rector rules to apply

@samsonasik samsonasik changed the title [Rector] Update to rector 0.12.14, apply parallel, and RemoveUnusedPrivatePropertyRector [Rector] Update to rector 0.12.15, apply parallel, and RemoveUnusedPrivatePropertyRector Jan 26, 2022
@samsonasik
Copy link
Contributor Author

updated to use rector 0.12.15

@samsonasik samsonasik closed this Jan 26, 2022
@samsonasik samsonasik reopened this Jan 26, 2022
@samsonasik
Copy link
Contributor Author

Close-Reopen to restart build

@samsonasik
Copy link
Contributor Author

All green 🎉

@butschster butschster merged commit 857948b into spiral:2.9 Jan 26, 2022
@samsonasik samsonasik deleted the update-rector-01213 branch January 26, 2022 18:08
@butschster
Copy link
Member

@butschster yes 👍 , just let me know there is a branch 3.0 to give php 8.0 rector rules to apply

I think, you can start from 2.9 branch

@samsonasik
Copy link
Contributor Author

2.9 branch still using php >=7.2:

"php": ">=7.2",

is it needs to be updated to >=8.0? For safer in the future, I think it is better to use:

"php": "~8.0.0 || ~8.1.0"

since php is not semver.

@butschster
Copy link
Member

2.9 branch still using php >=7.2:

"php": ">=7.2",

is it needs to be updated to >=8.0? For safer in the future, I think it is better to use:

"php": "~8.0.0 || ~8.1.0"

since php is not semver.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants