Skip to content

[Turbo] Fix Turbo CI (PHPStan & Functional tests) #2766

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

Open
wants to merge 1 commit into
base: 2.x
Choose a base branch
from

Conversation

Kocal
Copy link
Member

@Kocal Kocal commented May 23, 2025

Q A
Bug fix? no
New feature? no
Docs? no
Issues Fix #...
License MIT

Looks like the CI for Turbo was broken for a moment... let's fix it :)

@carsonbot carsonbot added Bug Bug Fix Feature New Feature Turbo Status: Needs Review Needs to be reviewed labels May 23, 2025
@Kocal Kocal force-pushed the phpstan-bootstrap branch 5 times, most recently from 13b3c53 to 98210ef Compare May 23, 2025 05:16
@Kocal Kocal changed the title [Turbo] Upgrade PHPStan to ^2.1.17, rename config file [Turbo] Upgrade PHPStan to ^2.1.17, rename PHPStan config file, and re-tweak CI May 23, 2025
@Kocal Kocal force-pushed the phpstan-bootstrap branch from 98210ef to 0fbed7e Compare May 23, 2025 05:21
@Kocal Kocal changed the title [Turbo] Upgrade PHPStan to ^2.1.17, rename PHPStan config file, and re-tweak CI [Turbo] Fix Turbo CI (PHPStan & Functional tests) May 23, 2025
@Kocal Kocal force-pushed the phpstan-bootstrap branch 2 times, most recently from d1389ea to b35b60a Compare May 23, 2025 05:59
@Kocal Kocal force-pushed the phpstan-bootstrap branch 2 times, most recently from 83d2504 to 4540f38 Compare May 23, 2025 06:24
Comment on lines -71 to -74
if (!\is_string($topic)) {
$topics[] = $topic;
continue;
}
Copy link
Member Author

Choose a reason for hiding this comment

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

I and PHPStan does not see how $topic can not be a string at this point.

Copy link
Member Author

Choose a reason for hiding this comment

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

friendly ping @rskuipers, by any chance :)

@Kocal Kocal removed Bug Bug Fix Feature New Feature labels May 23, 2025
@Kocal Kocal force-pushed the phpstan-bootstrap branch 2 times, most recently from fd9ac0d to 8936341 Compare May 23, 2025 06:30
@Kocal Kocal requested review from kbond and smnandre May 23, 2025 06:38
@Kocal Kocal force-pushed the phpstan-bootstrap branch 3 times, most recently from 6834cfc to 9cbfdf8 Compare May 23, 2025 07:54
@kbond
Copy link
Member

kbond commented May 23, 2025

Turbo tests still erroring?

@Kocal Kocal force-pushed the phpstan-bootstrap branch from 9cbfdf8 to c0d26ed Compare May 23, 2025 12:18
@Kocal
Copy link
Member Author

Kocal commented May 23, 2025

Turbo tests still erroring?

I did the fix in the wrong file 🤦🏻
image

@Kocal
Copy link
Member Author

Kocal commented May 23, 2025

However, sometimes we still have this flakky test:
image

It looks like it only happens on lowest deps, I will try to increase the version constraints of panther / webdriver
EDIT: didn't help :(

@Kocal Kocal force-pushed the phpstan-bootstrap branch from c0d26ed to df73091 Compare May 23, 2025 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review Needs to be reviewed Turbo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants