Skip to content

Update PHPStan to level 9#62

Merged
roberto-butti merged 3 commits intoHi-Folks:mainfrom
RoadSigns:feature/update-phpstan-to-level-9
Jan 7, 2023
Merged

Update PHPStan to level 9#62
roberto-butti merged 3 commits intoHi-Folks:mainfrom
RoadSigns:feature/update-phpstan-to-level-9

Conversation

@RoadSigns
Copy link
Contributor

Upgraded the PHPStan level to 9, there was no work required for the upgrade to level 8, so this was skipped.

It's part of an ongoing issue
#54

Resolving the following issues:

 ------ -----------------------------------------------------------------------------
  Line   Arr.php
 ------ -----------------------------------------------------------------------------
  309    Parameter #2 ...$arrays of function array_merge expects array, mixed given.
 ------ -----------------------------------------------------------------------------

@roberto-butti
Copy link
Contributor

This is pure Gold 🏅

Super job @RoadSigns
Going to merge!

@roberto-butti roberto-butti merged commit 6c634bd into Hi-Folks:main Jan 7, 2023
@RoadSigns RoadSigns deleted the feature/update-phpstan-to-level-9 branch January 10, 2023 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants