Closed (fixed)
Project:
Drupal core
Version:
10.1.x-dev
Component:
base system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
29 Mar 2023 at 14:00 UTC
Updated:
1 May 2023 at 06:34 UTC
Jump to comment: Most recent
Comments
Comment #3
spokjeComment #4
daffie commentedFor the 2 remarks on the PR.
Comment #5
spokjeComment #6
daffie commentedAll my remarks have been addressed.
For me it is RTBC.
Comment #7
spokjeRebased after #3351218: Fix PHPStan L1 errors "@covers value \Foo\Bar::baz() references an invalid method." was committed.
Comment #8
spokjeRebased after #3351073: Fix PHPStan L1 errors "Annotation "@covers::foo" is invalid, "@covers" should be followed by a space and a value." was committed.
NOTE: There's still one open thread in the MR that might(?) need some more input before committing.
Comment #9
longwaveComment #10
spokjeAddressed last open thread.
Comment #11
daffie commentedAll the changes look good to me.
Back to RTBC.
Comment #12
spokjeMerge blocked: fast-forward merge is not possible. To merge this request, first rebase locally.Comment #13
spokjeComment #14
spokjeThanks @daffie, back to RTBC after rebase.
Comment #17
longwaveCommitted and pushed to 10.1.x, thanks!