Problem/Motivation

Amongst the current suppressions found in the PHPStan level 1 baseline, introduce by the adding of phpstan/phpstan-phpunit, is: @dataProvider Foo related method not found..

This issue exists to fix all of those.

Steps to reproduce

- Run PHPStan on level 1 and see the above issue amongst all others.

Proposed resolution

- Solve all of the reported issues for the above mentioned.
- Run PHPStan on level 1 and don't see the above issue any more.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3351089

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Spokje created an issue. See original summary.

Spokje opened merge request !3747

spokje’s picture

spokje
Primary language Dutch
commented
Assigned: spokje » Unassigned
Status: Active » Needs review
mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Needs review » Reviewed & tested by the community

PHPStan recommended fixes, reducing the baseline. Thanks.

spokje’s picture

spokje
Primary language Dutch
commented

Rebased after #3351218: Fix PHPStan L1 errors "@covers value \Foo\Bar::baz() references an invalid method." was committed.

spokje’s picture

spokje
Primary language Dutch
commented

Rebased after #3351073: Fix PHPStan L1 errors "Annotation "@covers::foo" is invalid, "@covers" should be followed by a space and a value." was committed.

  • longwave committed 85aee30a on 10.1.x 
    Issue #3351089 by Spokje: Fix PHPStan L1 errors "@dataProvider Foo...
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed 85aee30af3 to 10.1.x. Thanks!

Spokje closed merge request !3747

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.