Skip to content

fix: 7119 parameter array shape uses invalid syntax #7151

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: 4.1
Choose a base branch
from

Conversation

nesl247
Copy link
Contributor

@nesl247 nesl247 commented May 15, 2025

Q A
Branch? 4.1
Tickets Closes #7119
License MIT
Doc PR N/A

Fixes the array syntax for Parameter::$schema. Similar to #7150.

@nesl247
Copy link
Contributor Author

nesl247 commented May 15, 2025

The tests are failing because I just realized api-platform isn't on phpstan 2. Is there a reason for that?

@soyuka soyuka mentioned this pull request May 16, 2025
@soyuka
Copy link
Member

soyuka commented May 16, 2025

right when phpstan 2.0 was released we couldn't as phpstan-prophecy wasn't updated but it looks fine now, let me try at #7154

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants