Skip to content

chore(deps): add rector - #2366

Merged
Antreesy merged 2 commits into
masterfrom
chore/rector
Oct 30, 2026
Merged

Antreesy merged 2 commits into
masterfrom
chore/rector

Conversation

@Antreesy

@Antreesy Antreesy commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator
  • Add rector with the PHP 8.3 and Nextcloud 35 sets in its own composer-bin namespace.
  • rector/rector is pinned to 2.7
  • Run and fix findings

⚠️ ReplaceInjectedMethodCallRector is skipped: rewrites to untyped IAppConfig::getValue()/setValue(); OCP only has typed getters/setters, so the type must be picked per call site

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (ClaudeCode:claude-opus-5-5)

@Antreesy Antreesy added this to the Nextcloud 36 milestone Oct 28, 2026
@Antreesy Antreesy self-assigned this Oct 28, 2026
@Antreesy
Antreesy added this pull request to stack #2367 October 28, 2026 16:54
Base automatically changed from chore/composer-bin to master October 29, 2026 13:21
@Antreesy
Antreesy marked this pull request as ready for review October 29, 2026 13:22
Add rector with the PHP 8.3 and Nextcloud 35 sets in its own
composer-bin namespace. rector/rector is pinned to 2.7, because
nextcloud/rector 0.5.2 is not compatible with rector 2.8.

ReplaceInjectedMethodCallRector is skipped, because it rewrites
IConfig app value calls to the untyped IAppConfig::getValue() and
setValue(). Those exist only in OC\AppConfig. OCP\IAppConfig has only
typed getters and setters, so the type must be picked per call site.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Generated with `composer rector:fix`.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy merged commit 73a93e3 into master Oct 30, 2026
27 checks passed
@Antreesy
Antreesy deleted the chore/rector branch October 30, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants