Skip to content

Conversation

@devbanana
Copy link
Contributor

Makes the types returned by SimpleXMLElement more specific. See the discussion here: phpstan/phpstan#5586

@ondrejmirtes
Copy link
Member

Alright, let's try this :) I'll revert it if some complications come up.

@ondrejmirtes ondrejmirtes merged commit 6986e0b into phpstan:master Sep 8, 2021
@ondrejmirtes
Copy link
Member

Thank you.

@devbanana devbanana deleted the fix-simplexmlelement-stubs branch September 8, 2021 17:12
@VincentLanglet
Copy link
Contributor

VincentLanglet commented Sep 15, 2021

This code is working fine on my project
https://phpstan.org/r/21751445-6289-4c07-8418-64ca4e71067e

But phpstan is now complaining since the last release.

What is the right way to do it ?

@ondrejmirtes
Copy link
Member

@VincentLanglet Please open an issue, discussion below a merged PR isn't the right place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants