Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Jan 5, 2026

Somehow this went missing in #5234

Also adds missing stub to Wrapper class for CI tests.

Signed-off-by: Julius Knorr <jus@bitgrid.net>
@juliusknorr juliusknorr requested a review from elzody as a code owner January 5, 2026 13:48
@juliusknorr juliusknorr requested a review from emberfiend January 5, 2026 13:49
@juliusknorr juliusknorr added bug Something isn't working 3. to review Ready to be reviewed labels Jan 5, 2026
Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards
Copy link
Member

Updated the stub for Wrapper; ci should pass now

@joshtrichards
Copy link
Member

Well I fixed the stub matter; now the new issue is psalm/box running against PHP 8.5. 🤦 :)

@elzody
Copy link
Contributor

elzody commented Jan 7, 2026

I don't understand why it's complaining about the PHP version -.-

@elzody
Copy link
Contributor

elzody commented Jan 7, 2026

I have a fix for that in a separate PR
#5258

@elzody
Copy link
Contributor

elzody commented Jan 7, 2026

Merging so that we have some CI fixed and can work on fixing other, unrelated CI failures.

@elzody elzody merged commit f153cac into main Jan 7, 2026
67 of 72 checks passed
@elzody elzody deleted the fix/favicon branch January 7, 2026 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Ready to be reviewed bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants