Skip to content

Add Initial Rector Base Configuration#35

Draft
AlexJump24 wants to merge 4 commits intomainfrom
feature/rector
Draft

Add Initial Rector Base Configuration#35
AlexJump24 wants to merge 4 commits intomainfrom
feature/rector

Conversation

@AlexJump24
Copy link
Collaborator

@AlexJump24 AlexJump24 commented May 30, 2023

Initial rector configuration that can be loaded into projects similar to ECS. PR open for discussion about what should/shouldn't be in there.

FinalizeClassesWithoutChildrenRector being particularly controversial, however like ECS these rules can be ignored at individual project level.

@AlexJump24 AlexJump24 changed the title WIP Add initial Rector Base Configuration May 30, 2023
@AlexJump24 AlexJump24 changed the title Add initial Rector Base Configuration Add Initial Rector Base Configuration May 30, 2023
@AlexJump24 AlexJump24 requested a review from a team May 30, 2023 11:17
$rectorConfig->rule(PrivatizeFinalClassPropertyRector::class);

//Type Declarations
$rectorConfig->rule(AddClosureReturnTypeRector::class);
Copy link
Contributor

Choose a reason for hiding this comment

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

😍

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