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.

Sign up for GitHub

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

Jump to bottom

Incorporate PHPStan for Static Analysis #6541

Closed
remyperona opened this issue Apr 4, 2024 · 0 comments · Fixed by #6617
Closed

Incorporate PHPStan for Static Analysis #6541

remyperona opened this issue Apr 4, 2024 · 0 comments · Fixed by #6617
Assignees
@remyperona
Labels
dev initiative type: enhancement Improvements that slightly enhance existing functionality and are fast to implement

Comments

@remyperona
Copy link
Contributor

PHPStan basic configuration has been added in #5995.

To go further, as a start we can:

  • Add the analysis to the CI lint action
  • Start fixing errors at level 0

Once this is done, we can open a new issue to fix level 1 and go on like this.
@remyperona remyperona added type: enhancement Improvements that slightly enhance existing functionality and are fast to implement dev initiative labels Apr 4, 2024
@remyperona remyperona self-assigned this May 6, 2024
@remyperona remyperona mentioned this issue May 6, 2024
Merged
6 tasks
github-merge-queue bot pushed a commit that referenced this issue May 8, 2024
@remyperona
Closes #6541 Add PHPStan CI & level 0 errors (#6617)
2b088a3
@remyperona remyperona mentioned this issue May 8, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@remyperona remyperona

Labels
dev initiative type: enhancement Improvements that slightly enhance existing functionality and are fast to implement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Closes #6541 Add PHPStan CI & level 0 errors wp-media/wp-rocket
1 participant
@remyperona