Skip to content

Conversation

@justlevine
Copy link
Collaborator

What does this implement/fix? Explain your changes.

This PR:

  • Adds phpstan/phpstan-deprecation-rules to the Composer dev-deps
  • Begins implementing bleeding edge PHPStan fixes

Both of these are part of the PHPStan 2.0 migration path. They're being dealt with here in preparation of the PHP version bump in #3247 which will allow us to upgrade the dependency.

Does this close any currently open issues?

Any relevant logs, error output, GraphiQL screenshots, etc?

Any other comments?

Further bleeding edge fixes will be implemented in future PRs.

Where has this been tested?

Operating System: Ubuntu 24.04 (wsl2 + ddev + php 8.2.26)

WordPress Version: 6.7.1

@justlevine justlevine requested a review from jasonbahl December 28, 2024 22:21
@justlevine justlevine added status: in review Awaiting review before merging or closing needs: reviewer response This needs the attention of a codeowner or maintainer type: chore Maintenance tasks, refactoring, and other non-functional changes scope: code quality Refactoring, linting, and enforcing coding standards labels Dec 28, 2024
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 195a7a6 and detected 0 issues on this pull request.

View more on Code Climate.

@coveralls
Copy link

Coverage Status

coverage: 83.99%. remained the same
when pulling 195a7a6 on justlevine:chore/phpstan-cleanup
into 61f5316 on wp-graphql:develop.

@jasonbahl jasonbahl merged commit 896ce77 into wp-graphql:develop Dec 31, 2024
36 checks passed
@justlevine justlevine deleted the chore/phpstan-cleanup branch December 31, 2024 16:36
@jasonbahl jasonbahl mentioned this pull request Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs: reviewer response This needs the attention of a codeowner or maintainer scope: code quality Refactoring, linting, and enforcing coding standards status: in review Awaiting review before merging or closing type: chore Maintenance tasks, refactoring, and other non-functional changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants