Skip to content

chore: Enable rector - #1018

Open
CarlSchwan wants to merge 6 commits into
masterfrom
carl/rector
Open

chore: Enable rector#1018
CarlSchwan wants to merge 6 commits into
masterfrom
carl/rector

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Comment thread tests/bootstrap.php Outdated
Comment thread composer.json Outdated
Comment thread rector.php
typeDeclarations: true,
typeDeclarationDocblocks: true,
privatization: true,
instanceOf: true,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it the one replacing !== null checks with instanceof? What is the incentive?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, it's this rule. I just copied the rules from server

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Enabled now that they are final

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
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