- Issue created by @tedbow
- Status changed to Needs review
over 1 year ago 4:45pm 25 May 2023 - last update
over 1 year ago 792 pass - @tedbow opened merge request.
- last update
over 1 year ago 790 pass, 1 fail - Status changed to RTBC
over 1 year ago 4:48pm 25 May 2023 - πΊπΈUnited States tedbow Ithaca, NY, USA
verbal RTBC from phΓ©naproxima. Maybe he wants deniability?
- last update
over 1 year ago 792 pass - last update
over 1 year ago 792 pass - last update
over 1 year ago 792 pass -
tedbow β
committed e939f093 on 3.0.x
Issue #3362746 by tedbow: fix 10.0.x phpstan errors
-
tedbow β
committed e939f093 on 3.0.x
- Assigned to tedbow
- Status changed to Needs work
over 1 year ago 9:32am 26 May 2023 - π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Huh? What happened here? This is NOT just fixing PHPStan errors. This de facto committed π Assert no errors after creating the test project in ModuleUpdateTest Needs review ?! π³
I've been asking to fix that for 4 months, and now it's been done here? I've surfaced this many times over the past month in meetings.
How is this related to PHPStan?
- πΊπΈUnited States tedbow Ithaca, NY, USA
@Wim Leers sorry.
See the
* @phpstan-ignore-next-line
in the commit.this was causing the code check fail on drupalci. when it was fixed it showed 10.0.7 failed.
I didn't remember there was already an issue for this in π Assert no errors after creating the test project in ModuleUpdateTest Needs review . I am still not clear if that issue was causes test to fail at the time it was created. I believe it cause test to fail on 10.0.x not because drupalci was testing on specific version of core not the 10.0.x dev branch.
We can revert this and then just commit the phpstan fix. But then test will still be failing on 10.0.7
We will wait till we talk though
- Status changed to Fixed
about 1 year ago 2:21am 3 November 2023 Automatically closed - issue fixed for 2 weeks with no activity.