Skip to content

Larastan Updates #26

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

shawnhooper
Copy link
Contributor

  1. Larastan has moved from nunomaduro/larastan to larastan/larastan
  2. Running vendor/bin/phpstan analyze src/ found 4 potential Unsafe usage of new static() errors

@shawnhooper
Copy link
Contributor Author

I updated the initial PR to include a phpstan configuration that runs tests at level 2. Without a specified level, it was running at level 0. I fixed the errors so that level 2 would pass.

More errors appear as you increase each level from 3 to 8.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant