Skip to content

[4.x] Add PHPStan to test environment with max level - #76

Merged
SimonFrings merged 2 commits into
reactphp:4.xfrom
clue-labs:phpstan-v4
Jun 14, 2023
Merged

[4.x] Add PHPStan to test environment with max level#76
SimonFrings merged 2 commits into
reactphp:4.xfrom
clue-labs:phpstan-v4

Conversation

@clue

@clueclue commented Jun 13, 2023

Copy link
Copy Markdown
Member

This changeset adds PHPStan on max level to the test environment for all supported PHP versions. Unlike #64 runs the maximum level supported with all errors addressed without having to resort to a baseline. The changeset size seems reasonable and this does not other affect our public API, so this should be safe to apply.

If this PR is merged, I will backport the changes from v4 to v3. The changes are almost identical across all versions, but only v4 takes advantage of fibers (#15).

Builds on top of clue/framework-x#200, clue/framework-x#201 and #66
See also https://github.com/orgs/reactphp/discussions/469 and #40

@clueclue added this to the v4.1.0 milestone Jun 13, 2023
Comment threadtests/AsyncTest.php Outdated
Comment threadphpstan.neon.dist
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@clue@WyriHaximus@SimonFrings