Skip to content

Conversation

@weirdan
Copy link
Collaborator

@weirdan weirdan commented Jan 30, 2024

  • key_exists() is an alias for array_key_exists()
  • Allow properties on intersections with enum interfaces
  • readgzfile() is impure
  • Do not validate callable arguments in lenient contexts
  • [LSP] Add issue type in description
  • Re-work CheckTrivialExprVisitor
  • Fix language server running with opcache.save_comments=0
  • Report MissingConstructor for natively typed mixed properties
  • Allow importing typedefs from enums

weirdan and others added 21 commits January 27, 2024 13:41
…tion

[LSP] Add issue type in description
In php-parser 5.0, `ClosureUse` is no longer considered an expression.
This requires changes to Psalm's `CheckTrivialExprVisitor`, which stores
an array of "non-trivial" `Expr` nodes.

However the only use of this array is to count whether or not it's
empty. Instead of keeping the array, we can keep a boolean, and avoid
needing to change the types in this class when we upgrade to php-parser
5.0.
@weirdan weirdan added the release:internal The PR will be included in 'Internal changes' section of the release notes label Jan 30, 2024
@weirdan weirdan requested review from danog and orklah January 30, 2024 21:08
@weirdan weirdan marked this pull request as ready for review January 30, 2024 21:08
@edsrzf
Copy link

edsrzf commented Jan 30, 2024

Any chance we could merge #10605 before merging this, so I can rebase #10567 onto all the parts I've extracted?

@weirdan weirdan merged commit bf57d59 into vimeo:master Jan 30, 2024
@weirdan weirdan deleted the update-master branch January 30, 2024 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:internal The PR will be included in 'Internal changes' section of the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants