Skip to content

chore/rector: make phpunit dataproviders phpunit 12 compatible #53728

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.

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

Merged
merged 3 commits into from
Jul 2, 2025

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
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
@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
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.

4 participants