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

PHP 8.0: verified support for "mixed" type declarations #163

Merged
merged 3 commits into from
Jul 10, 2020

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Jul 4, 2020

Added tests to the following methods to verify that the new mixed type declaration is handled correctly (hint: it is).

  • BCFile::getMethodParameters()
  • FunctionDeclarations::getParameters() (via BCFile::getMethodParameters())
  • BCFile::getMethodProperties()
  • FunctionDeclarations::getProperties() (via BCFile::getMethodProperties())
  • BCFile::getMemberProperties()
  • Variables::getMemberProperties() (via BCFile::getMemberProperties())

@jrfnl jrfnl added this to the 1.0.0-alpha4 milestone Jul 4, 2020
@jrfnl jrfnl force-pushed the feature/php8-mixed-type-add-unit-tests branch 2 times, most recently from 3260633 to 2470e36 Compare July 5, 2020 18:10
…PHP 8 "mixed" type

No changes needed to the actual method, the parameter type is already handled correctly.
…PHP 8 "mixed" return type

No changes needed to the actual method, the return type is already handled correctly.
… property type

No changes needed to the actual method, the type is already handled correctly.
@jrfnl jrfnl force-pushed the feature/php8-mixed-type-add-unit-tests branch from 2470e36 to cf89158 Compare July 10, 2020 05:37
@jrfnl jrfnl merged commit a07fd2e into develop Jul 10, 2020
@jrfnl jrfnl deleted the feature/php8-mixed-type-add-unit-tests branch July 10, 2020 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant