Skip to content

refactor: Thumbnail Generator logging and tests - #52299

Merged
susnux merged 1 commit into
masterfrom
refactor/preview-tests
Apr 24, 2025
Merged

refactor: Thumbnail Generator logging and tests#52299
susnux merged 1 commit into
masterfrom
refactor/preview-tests

Conversation

@nfebe

Copy link
Copy Markdown
Contributor

Minor follow up for : #52221

@nfebenfebe added 3. to review Waiting for reviews ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Apr 22, 2025
@nfebe
nfebe requested a review from a team as a code ownerApril 22, 2025 11:07
@github-project-automationgithub-project-automationBot moved this to 🏗️ In progress in 📁 Files teamApr 22, 2025
@provokateurin

Copy link
Copy Markdown
Member

The refactored tests seem a bit unrelated? At least there are no actual changes to the tests afaict.

@nfebe

Copy link
Copy Markdown
ContributorAuthor

The refactored tests seem a bit unrelated? At least there are no actual changes to the tests afaict.

Yes, it just a refactor, as I opened those files but the time I could amend the commit the failing jobs passed for the main changes so moved this to a new PR

Comment threadtests/lib/Preview/Provider.php Outdated
@nfebe
nfebeforce-pushed the refactor/preview-tests branch from cf54e33 to fd26bafCompareApril 22, 2025 21:46
Comment threadtests/lib/Preview/Provider.php Outdated
@nfebe
nfebeforce-pushed the refactor/preview-tests branch 2 times, most recently from a985e2b to 122cd46CompareApril 23, 2025 21:39
Signed-off-by: nfebe <fenn25.fn@gmail.com>
@nfebe
nfebeforce-pushed the refactor/preview-tests branch from 122cd46 to d980e69CompareApril 23, 2025 22:06
@susnux
susnux merged commit d86189a into masterApr 24, 2025
@susnux
susnux deleted the refactor/preview-tests branch April 24, 2025 07:50
@nfebe

Copy link
Copy Markdown
ContributorAuthor

/backport to stable31

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

Labels

3. to reviewWaiting for reviews♻️ refactorRefactor code (not a bug fix, not a feature just refactoring)

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

4 participants

@nfebe@provokateurin@susnux@come-nc