Skip to content

refactor: Extend rector to all top-level files #52853

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.

Sign up for GitHub

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

Jump to bottom
Merged
provokateurin merged 2 commits into from
May 15, 2025
Merged

refactor: Extend rector to all top-level files #52853

provokateurin merged 2 commits into from
May 15, 2025
+62 −29

Conversation

@provokateurin
Copy link
Member

@provokateurin provokateurin commented May 15, 2025

Summary

Checklist

@provokateurin provokateurin added this to the Nextcloud 32 milestone May 15, 2025
@provokateurin provokateurin requested review from come-nc and susnux May 15, 2025 08:34
@provokateurin provokateurin requested a review from a team as a code owner May 15, 2025 08:34
@provokateurin provokateurin requested review from artonge and yemkareems and removed request for a team May 15, 2025 08:34
@provokateurin provokateurin added 3. to review Waiting for reviews ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels May 15, 2025
@provokateurin provokateurin enabled auto-merge May 15, 2025 09:47
@provokateurin provokateurin mentioned this pull request May 15, 2025
Merged
4 tasks
come-nc
come-nc approved these changes May 15, 2025
View reviewed changes
Copy link
Contributor

@come-nc come-nc left a comment

Choose a reason for hiding this comment

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

👍

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 15, 2025
@provokateurin provokateurin merged commit 701fcbf into master May 15, 2025
201 of 208 checks passed
@provokateurin provokateurin deleted the refactor/rector-top-level branch May 15, 2025 10:41
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
Merged
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@icewind1991 icewind1991 icewind1991 approved these changes

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@come-nc come-nc come-nc approved these changes

@susnux susnux Awaiting requested review from susnux

@artonge artonge Awaiting requested review from artonge artonge is a code owner automatically assigned from nextcloud/server-backend

@yemkareems yemkareems Awaiting requested review from yemkareems yemkareems is a code owner automatically assigned from nextcloud/server-backend

Assignees

No one assigned

Labels

4. to release Ready to be released and/or waiting for tests to finish ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring)

Projects

None yet

Milestone

Nextcloud 32

Development

Successfully merging this pull request may close these issues.

6 participants

@provokateurin @icewind1991 @ChristophWurst @come-nc @skjnldsv