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 master from 5.x #10656

Merged
merged 37 commits into from
Feb 5, 2024
Merged

Update master from 5.x #10656

merged 37 commits into from
Feb 5, 2024

Conversation

weirdan
Copy link
Collaborator

@weirdan weirdan commented Feb 5, 2024

  • Fix ownerDocument type
  • Fix Assumed filter_var return type can be incorrect #10561 numeric input incorrect return type
  • add scalar & numeric handling for all cases where appropriate and ensure no more generic types being added for int/float (previous commit)
  • Allow inline comments in typedef shapes
  • Allow typedef imports from any classlike type
  • Fix baseline loading for path specified on the command line
  • Stable baseline
  • Allow enum cases to be global constants
  • CallMap: Adjust return type for inotify_add_watch() to int|false
  • CallMap: Improve returntype annotation for inotify_read()
  • Allow sebastian/diff v6
  • Update our actual baseline to use the new format
  • Drop unused local composer repo
  • Fix check-type when using reserved types from within a namespace
  • Support user defined types for psalm-check-type
  • Add InvalidOverride issue
  • Emit InvalidOverride
  • Clarify that Pull request labels failure is to be resolved by maintainers
  • Do not add callable as a native property type
  • Allow adding Closure as a native property type

fluffycondor and others added 30 commits January 31, 2024 16:10
@fluffycondor
Fix ownerDocument type
6aa2ddf
@weirdan
Merge pull request vimeo#10619 from fluffycondor/fix-ownerdocument-null
abe7ae8
@kkmuffme
Fix vimeo#10561 numeric input incorrect return type
551625a
@kkmuffme
add scalar & numeric handling for all cases where appropriate and ens…
7023855 
…ure no more generic types being added for int/float (previous commit)
@weirdan
Merge pull request vimeo#10621 from kkmuffme/fix-numeric-scalar-valid…
215d62e 
…ate-filter-var-input
@weirdan
Allow inline comments in typedef shapes
9ec556b 
Fixes vimeo#10492
@weirdan
Merge pull request vimeo#10623 from weirdan/allow-inline-comments-in-…
4fdfbac 
…typedef-shapes
@weirdan
Allow typedef imports from any classlike type
6140f68 
All we really need is for the source to be autoloadable, and it includes
all classlikes (interfaces, classes, enums and traits at the time of
writing).
@weirdan
Fix baseline loading for path specified on the command line
138c7b8 
Fixes vimeo#10624
@weirdan
Merge pull request vimeo#10628 from weirdan/fix-baseline-usage-with-p…
8c473e2 
…aths-passed-via-cli-args
@weirdan
Merge branch '5.21.x' into 5.x
0082207
@weirdan
Stable baseline
76364ab 
Fixes vimeo#10632
@weirdan
Merge pull request vimeo#10625 from weirdan/allow-typedef-imports-fro…
38d7d43 
…m-any-kind-of-classlike
@robchett @weirdan
Allow enum cases to be global constants
421cb0f
@weirdan
Merge pull request vimeo#10634 from weirdan/10334-global-constants-as…
fc88ef2 
…-case-values
@UlrichEckhardt
CallMap: Adjust return type for inotify_add_watch() to int|false
1698239 
See vimeo#10636
@UlrichEckhardt
CallMap: Improve returntype annotation for inotify_read()
9fd17cf 
In case of success, it returns an array of associative arrays with defined fields:
- `wd` is a watch descriptor
- `mask` is a bit mask of events
- `cookie` is a unique id to connect related events
- `name` is the name of a file
@simPod
Allow sebastian/diff v6
4fc3594
@weirdan
Update our actual baseline to use the new format
9a970ca
@weirdan
Merge pull request vimeo#10633 from weirdan/stable-baseline
192f7b9
@weirdan
Drop unused local composer repo
a2980b5 
This was intended to show how you could convert legacy plugins to
composer format, but it breaks `composer install` in snapshots.
@robchett
Fix check-type when using reserved types from within a namespace
526013e
@robchett
Support user defined types for psalm-check-type
9d4fd40
@weirdan
Merge pull request vimeo#10647 from weirdan/drop-echo-checker-repo
c1400b8
@weirdan
Merge pull request vimeo#10639 from simPod/diff
90362df
@weirdan
Merge pull request vimeo#10637 from UlrichEckhardt/fix/issue-10636-in…
74314eb 
…otify-returntype
@weirdan
Merge pull request vimeo#10648 from robchett/check_type_namespace_all…
080c8f6 
…ow_reserved_words
@edsrzf
Add InvalidOverride issue
07f4045
@edsrzf
Emit InvalidOverride
ea9cb44
@weirdan
Merge pull request vimeo#10644 from edsrzf/invalid-override
3b8d81b
@weirdan
Clarify that Pull request labels failure is to be resolved by maintai…
7c7c216 
…ners
@weirdan
Merge pull request vimeo#10649 from weirdan/ci-clarify-pull-request-l…
59df6a7 
…abels-check

Clarify that Pull request labels failure is to be resolved by maintainers
@weirdan
Do not add callable as a native property type
e8b47f7 
It's invalid in all PHP versions: https://3v4l.org/bXWo2

Also see php.net/manual/en/language.types.declarations.php#language.types.declarations.base.function

Fixes vimeo#10650
@weirdan
Allow adding Closure as a native property type
b2a2cd7
@weirdan
Merge pull request vimeo#10654 from weirdan/10650-do-not-add-callable…
9ac2383 
…-as-native-property-type
@weirdan
Merge branch '5.x' into update-master
7bfdf7d
@weirdan weirdan added the release:internal The PR will be included in 'Internal changes' section of the release notes label Feb 5, 2024
@weirdan weirdan requested review from danog and orklah February 5, 2024 03:47
@weirdan weirdan marked this pull request as ready for review February 5, 2024 03:57
@weirdan weirdan merged commit 4b2c698 into vimeo:master Feb 5, 2024
41 of 42 checks passed
@weirdan weirdan deleted the update-master branch February 5, 2024 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danog danog danog approved these changes

@orklah orklah Awaiting requested review from orklah

Assignees
No one assigned
Labels
release:internal The PR will be included in 'Internal changes' section of the release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@weirdan @danog @fluffycondor @kkmuffme @robchett @UlrichEckhardt @simPod @edsrzf