Skip to content

Conversation

@icewind1991
Copy link
Member

phpunit removed the non-static dataproviders and the docblock based annotation for them

@icewind1991 icewind1991 added this to the Nextcloud 32 milestone Jun 30, 2025
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Jun 30, 2025
@icewind1991 icewind1991 requested review from Altahrim, artonge and sorbaugh and removed request for a team June 30, 2025 14:57
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Reviewed every single line changed /s

@tcitworld
Copy link
Member

Is there any way to import attribute namespaces?

@ChristophWurst
Copy link
Member

There is a rule for rector to auto import but it will run on all code, not just the changed bits :s

@icewind1991 icewind1991 added the ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) label Jun 30, 2025
@icewind1991 icewind1991 merged commit 2337bd8 into master Jul 2, 2025
203 of 205 checks passed
@icewind1991 icewind1991 deleted the rector-phpunit10 branch July 2, 2025 16:30
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
@skjnldsv skjnldsv added this to the Nextcloud 32 milestone Oct 2, 2025
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 ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants