- Status changed to Needs work
almost 2 years ago 2:49pm 21 January 2023 - ๐ณ๐ฑNetherlands spokje
Discussed this with @mondrake in Slack, our plan is to:
- Use ๐ DrupalStandardsListenerTrait is broken Closed: won't fix to fix
DrupalStandardsListenerTrait
and all the@covers
errors it detects once fixed.
- Then return to this issue and try to getphpstan/phpstan-phpunit
added as a dev dependency.
- Since we then have coverage of@covers
(Baduum tish, I'm here all week, try the veal...) throughphpstan/phpstan-phpunit
we can then deprecate/removeDrupalStandardsListenerTrait
in (Yet Another) issue to be more ready for PHPUnit 10 that drops the usage of listeners.So, for now, postponing this issue on ๐ DrupalStandardsListenerTrait is broken Closed: won't fix .
- Status changed to Postponed
almost 2 years ago 7:26pm 21 January 2023 - ๐ฎ๐นItaly mondrake ๐ฎ๐น
Rebased and fixed more errors that show up in code committed recently.
- ๐ฎ๐นItaly mondrake ๐ฎ๐น
Rebased and fixed more errors that show up in code committed recently.
- Status changed to Needs work
over 1 year ago 3:05pm 28 March 2023 - First commit to issue fork.
- Status changed to Needs review
over 1 year ago 4:41pm 28 March 2023 - Assigned to mondrake
- Status changed to Needs work
over 1 year ago 5:39pm 28 March 2023 - ๐ฎ๐นItaly mondrake ๐ฎ๐น
Thanks. As much as I hate the idea, I think we should better split the issue: here just the added dependency with baselined/ignored errors, and in followups the error fixes and baseline cleanup. On that.
- @mondrake opened merge request.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 8:03pm 28 March 2023 - ๐ณ๐ฑNetherlands spokje
Rebasing to be able to retest and get rid of the (alarmingly increased) random JS test failures.
- Status changed to RTBC
over 1 year ago 7:35am 29 March 2023 - ๐ณ๐ฑNetherlands spokje
Right....
Random JS test failures galore!
Not wasting any more test-cycles on that, marking RTBC.
- Status changed to Fixed
over 1 year ago 10:46am 29 March 2023 -
alexpott โ
committed 947fd48d on 10.1.x
Issue #3326239 by mondrake, Spokje, andypost, longwave, xjm, quietone:...
-
alexpott โ
committed 947fd48d on 10.1.x
- ๐ซ๐ทFrance andypost
@mondrake one follow-up to clean-up baseline should be enough?
- ๐ณ๐ฑNetherlands spokje
Opened ๐ [META] Fix PHPStan baseline suppressions introduced with phpstan/phpstan-phpunit Fixed to get rid of the new suppressions in the PHPStan baseline.
- ๐ฌ๐งUnited Kingdom alexpott ๐ช๐บ๐
I created a CR for this https://www.drupal.org/node/3351386 โ
Automatically closed - issue fixed for 2 weeks with no activity.