Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Bring Project and CI Up-to-date #89

Merged
merged 10 commits into from
Apr 2, 2024
Merged

Bring Project and CI Up-to-date #89

merged 10 commits into from
Apr 2, 2024

Conversation

zanbaldwin
Copy link
Member

  • Test against PHP versions 5.6 to 8.3
  • Use PHPStan on max level

@zanbaldwin zanbaldwin force-pushed the feature/phpstan branch 2 times, most recently from e77913a to 4e5e950 Compare March 28, 2024 14:36
So that people can easily see that static analysis is used, even if they aren't mentioned in Composer's require-dev.
I have much more experience with PHPStan than with Psalm. Stick to what you know best.
Ignore the errors that only occur because we have guard clauses for lower PHP versions
@zanbaldwin zanbaldwin force-pushed the feature/phpstan branch 2 times, most recently from b0bbdc2 to 4204037 Compare April 2, 2024 08:47
@zanbaldwin zanbaldwin merged commit 13f29ae into develop Apr 2, 2024
21 checks passed
@zanbaldwin zanbaldwin deleted the feature/phpstan branch April 2, 2024 09:45
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.

1 participant