- Issue created by @Spokje
- last update
10 months ago 30,058 pass - @spokje opened merge request.
- Status changed to Needs review
10 months ago 10:16am 25 August 2023 - Status changed to RTBC
10 months ago 2:48pm 25 August 2023 - Status changed to Needs work
10 months ago 3:09pm 25 August 2023 - π¬π§United Kingdom catch
Will need a re-roll after π Bump mglaman/phpstan-drupal to latest to make daily "updated deps" QA run pass again Fixed
- last update
10 months ago Custom Commands Failed - last update
10 months ago 30,058 pass - Status changed to RTBC
10 months ago 4:05pm 25 August 2023 - π³π±Netherlands Spokje
Rerolled with PHPStan baseline suppression removed and re-RTBC-ed, since there were no code changes.
- last update
10 months ago 30,060 pass - Status changed to Fixed
10 months ago 8:25am 26 August 2023 - π¬π§United Kingdom catch
The class still implements the method so from that point of view it's not a bc break. The class no longer implements the interface - this would be a problem if these were supposed to be type hinted anywhere, but they're only used by http bridge so I think that's OK too (although I hope I'm not wrong).
However, do we need a follow-up to deprecate and then remove methods like Psr7RequestValueResolver::supports() in 11.x? Should be cruft now.
- π³π±Netherlands Spokje
Follow-up in π Deprecate Psr7RequestValueResolver::supports and RouteMatchValueResolver::supports Fixed .
Automatically closed - issue fixed for 2 weeks with no activity.