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

Update PHPStan to 1.0, rector to 0.12.x #4874

Merged
merged 7 commits into from Nov 10, 2021
Merged

Update PHPStan to 1.0, rector to 0.12.x #4874

merged 7 commits into from Nov 10, 2021

Conversation

staabm
Copy link
Member

@staabm staabm commented Nov 1, 2021 •
edited

XXX hier gehts erst weiter wenn es ein phpstan 1.0 kompatibles rector release gibt (jetzt weiß ich auch warum es von dependabot nicht als update vorgeschlagen wurde)

@gharlan
Copy link
Member

gharlan commented Nov 9, 2021 •
edited

Den Comparison-Fehler, den man hier im Diff sieht, bekomme ich auf phpstan.org nicht reproduziert:
https://phpstan.org/r/7c4b63d3-43f9-44f4-ab17-15f17df996de

Wurde das eventuell schon gefixt?

Und komisch ist noch, dass ich lokal noch einen weiteren Fehler bekomme, der hier auf github aber nicht zu sehen ist:

Bildschirmfoto 2021-11-09 um 18 25 34

Ich habe extra vorher lokal nochmal den Result-Cache von phpstan geleert.

@staabm
Copy link
Member Author

staabm commented Nov 9, 2021

Den Comparison-Fehler, den man hier im Diff sieht, bekomme ich auf phpstan.org nicht reproduziert:
https://phpstan.org/r/7c4b63d3-43f9-44f4-ab17-15f17df996de

Du kannst lokal mit PHPStan\dumpType($expire); ausgeben kassen, welchen type phpstan der variable zuordnet. Die ausgabe ist im phpstan analysis result zu sehen

@staabm
Copy link
Member Author

staabm commented Nov 9, 2021

Repro ist vermutlich https://phpstan.org/r/32e90791-101d-4ae5-b9eb-dd26c378c867

@staabm
Copy link
Member Author

staabm commented Nov 10, 2021

Ohh.. gleiches problem von nem anderen user gemeldet phpstan/phpstan#5969 :)

@staabm staabm marked this pull request as ready for review November 10, 2021 19:43
@staabm staabm changed the title Update PHPStan to 1.0 Update PHPStan to 1.0, rector to 0.12.x Nov 10, 2021
@staabm staabm added the automerge Automatisch PR rebasen und mergen label Nov 10, 2021
@kodiakhq kodiakhq bot merged commit 650dd65 into main Nov 10, 2021
@kodiakhq kodiakhq bot deleted the staabm-patch-2 branch November 10, 2021 19:46
@staabm
Copy link
Member Author

staabm commented Nov 10, 2021

Danke Gregor

@rex-bot rex-bot removed the automerge Automatisch PR rebasen und mergen label Nov 10, 2021
BenJ1337 pushed a commit to BenJ1337/redaxo that referenced this pull request Mar 6, 2022
@staabm @gharlan @BenJ1337
Update PHPStan to 1.0, rector to 0.12.x (redaxo#4874)
48a465d 
Co-authored-by: Gregor Harlan <330436+gharlan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@staabm @gharlan @rex-bot