Skip to content

[Task]: Require PHPStan v2.0#17816

Closed
kingjia90 wants to merge 7 commits into11.5from
phpstan-2
Closed

[Task]: Require PHPStan v2.0#17816
kingjia90 wants to merge 7 commits into11.5from
phpstan-2

Conversation

@kingjia90
Copy link
Copy Markdown
Contributor

PHPStan v2 just got released, let's see if there's any breaking change

https://github.com/phpstan/phpstan/releases/tag/2.0.0

@github-actions
Copy link
Copy Markdown

Review Checklist

  • Target branch (11.4 for bug fixes, others 11.x)
  • Tests (if it's testable code, there should be a test for it - get help)
  • Docs (every functionality needs to be documented, see here)
  • Migration incl. install.sql (e.g. if the database schema changes, ...)
  • Upgrade notes (deprecations, important information, migration hints, ...)
  • Label
  • Milestone

@kingjia90
Copy link
Copy Markdown
Contributor Author

kingjia90 commented Nov 11, 2024

Might be depending on ergebnis/phpstan-rules#859

Update: a PR seems there ergebnis/phpstan-rules#873

@ghost ghost added the Pimcore:ToDo label Nov 19, 2024
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Dec 3, 2024

@kingjia90
Copy link
Copy Markdown
Contributor Author

kingjia90 commented Dec 3, 2024

It seems more strict now (instanceof checks, unused traits) and that there are many rules that were previously ignored or things like checking PHPDocs etc..
https://github.com/phpstan/phpstan/blob/2.0.x/UPGRADING.md#noteworthy-changes-to-code-analysis

@alexz707 alexz707 changed the base branch from 11.4 to 11.5 December 17, 2024 11:29
@kingjia90 kingjia90 closed this Jan 29, 2025
@kingjia90 kingjia90 deleted the phpstan-2 branch January 29, 2025 08:48
@github-actions github-actions bot locked and limited conversation to collaborators Jan 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant