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

Fixed PHP 7.4 related issues #1957

Merged
merged 1 commit into from
Jul 19, 2019
Merged

Conversation

weirdan
Copy link
Collaborator

@weirdan weirdan commented Jul 19, 2019

  • s/fn/f/g (fn is a reserved keyword in 7.4)
  • ReflectionType::__toString() warning (this method is deprecated in 7.4)

Note that some issues are caused by phpspec/propecy code and will be fixed when phpspec/prophecy#432 lands. Most were caused by ReflectionType::__toString() usage in
Psalm\Internal\Codebase\Reflection though.

- s/fn/f/g (fn is a reserved keyword in 7.4)
- ReflectionType::__toString() warning (this method is deprecated in 7.4)

Note that some issues are caused by `phpspec/propecy` code and will be
fixed when phpspec/propecy#432 lands. Most were caused by
`ReflectionType::__toString()` usage in
`Psalm\Internal\Codebase\Reflection` though.
@muglug muglug merged commit e0a59ce into vimeo:master Jul 19, 2019
@muglug
Copy link
Collaborator

muglug commented Jul 19, 2019

Thanks!

jinyongyoo pushed a commit to jinyongyoo/psalm that referenced this pull request Jul 24, 2019
- s/fn/f/g (fn is a reserved keyword in 7.4)
- ReflectionType::__toString() warning (this method is deprecated in 7.4)

Note that some issues are caused by `phpspec/propecy` code and will be
fixed when phpspec/propecy#432 lands. Most were caused by
`ReflectionType::__toString()` usage in
`Psalm\Internal\Codebase\Reflection` though.
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.

2 participants