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.

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

Update master from 5.x #10618

Merged
merged 23 commits into from
Jan 30, 2024
Merged

Update master from 5.x #10618

merged 23 commits into from
Jan 30, 2024

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
Contributor

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
42 checks passed
@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.

None yet

2 participants