Skip to content

chore: install rector#3405

Open
Chartman123 wants to merge 1 commit into
mainfrom
chore/rector
Open

chore: install rector#3405
Chartman123 wants to merge 1 commit into
mainfrom
chore/rector

Conversation

@Chartman123

@Chartman123 Chartman123 commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Install rector and use the following rules/sets:

  • NEXTCLOUD_32
  • php81

Signed-off-by: GitHub noreply@github.com

@Chartman123 Chartman123 self-assigned this Jun 10, 2026
@Chartman123 Chartman123 changed the title chore: install rector and run it with empty ruleset chore: install rector Jun 10, 2026
@Chartman123 Chartman123 force-pushed the chore/rector branch 4 times, most recently from 7f4413c to 7604615 Compare June 10, 2026 20:32
Signed-off-by: Christian Hartmann <chris-hartmann@gmx.de>
@Chartman123 Chartman123 marked this pull request as ready for review June 10, 2026 20:58
@Chartman123 Chartman123 added php PHP related ticket 3. to review Waiting for reviews technical debt Technical issue labels Jun 10, 2026
@Koc

Koc commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

wow, nice! Do we want add it into pipeline?

@Chartman123

Copy link
Copy Markdown
Collaborator Author

There's a workflow template here that runs rector once a week and creates a PR with the resulting changes.

https://github.com/nextcloud/.github/blob/master/workflow-templates%2Frector-apply.yml

However I didn't find an org template for adding it to the CI pipeline

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews php PHP related ticket technical debt Technical issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants