Skip to content

fix(phpunit10): Migrate away from PHPUnit at() calls - #51768

Merged
nickvergessen merged 3 commits into
masterfrom
techdebt/noid/prepare-phpunit10
Mar 31, 2025
Merged

fix(phpunit10): Migrate away from PHPUnit at() calls#51768
nickvergessen merged 3 commits into
masterfrom
techdebt/noid/prepare-phpunit10

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

nickvergessenand others added 2 commits March 27, 2025 23:23
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@nickvergessennickvergessen added bug 3. to review Waiting for reviews labels Mar 27, 2025
@nickvergessennickvergessen added this to the Nextcloud 32 milestone Mar 27, 2025
@nickvergessennickvergessen self-assigned this Mar 27, 2025
@nickvergessen
nickvergessen requested a review from a team as a code ownerMarch 27, 2025 22:27
@nickvergessen
nickvergessen requested review from come-nc, icewind1991, nfebe and susnux and removed request for a teamMarch 27, 2025 22:27
@susnux

This comment was marked as resolved.

@nickvergessen
nickvergessenforce-pushed the techdebt/noid/prepare-phpunit10 branch from f0b2153 to 8bc21eeCompareMarch 28, 2025 11:52
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the techdebt/noid/prepare-phpunit10 branch from 8bc21ee to 522be60CompareMarch 31, 2025 07:43

@come-nccome-nc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would feel better if we can avoid expecting a specific default value for config keys, otherwise good job!

Comment threadtests/lib/Template/JSCombinerTest.php
@nickvergessen
nickvergessen merged commit aee2f2d into masterMar 31, 2025
@nickvergessen
nickvergessen deleted the techdebt/noid/prepare-phpunit10 branch March 31, 2025 14:04
@nextcloud-botnextcloud-bot mentioned this pull request Aug 19, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@nickvergessen@susnux@come-nc@skjnldsv