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

1.0.6 - Support older version of BetterReflection #32

Merged
merged 3 commits into from
Nov 14, 2018

Conversation

janmartenjongerius
Copy link
Contributor

Version 3 of BetterReflection requires PHP parser 4, which in turn is
also required by PHPStan. PHPStan has a dependency on Symfony Console,
which supports 3 | 4 in more recent versions, and goes back to 2 in
older versions, which in turn force PHP parser 3 and thus becomes
incompatible with the newer version of BetterReflection.

With the new changes, Magento 2 projects and modules can be statically
analyzed by DependencyGuard.

Jan-Marten de Boer added 3 commits October 16, 2018 09:05
* 'master' of github.com:mediact/dependency-guard:
  Increase memory limit for CI environment
  Fix #25 - Out of memory errors
  Add regression test for issue #25
* 'master' of github.com:mediact/dependency-guard:
  Be more specific on when the class in the catch is autoloaded
Version 3 of BetterReflection requires PHP parser 4, which in turn is
also required by PHPStan. PHPStan has a dependency on Symfony Console,
which supports 3 | 4 in more recent versions, and goes back to 2 in
older versions, which in turn force PHP parser 3 and thus becomes
incompatible with the newer version of BetterReflection.

With the new changes, Magento 2 projects and modules can be statically
analyzed by DependencyGuard.
@janmartenjongerius janmartenjongerius added bug Something isn't working verified Verified reported behavior labels Nov 14, 2018
@janmartenjongerius janmartenjongerius self-assigned this Nov 14, 2018
@janmartenjongerius janmartenjongerius requested review from michielfb and removed request for ksangers November 14, 2018 13:45
@janmartenjongerius janmartenjongerius merged commit a82b192 into master Nov 14, 2018
@ksangers ksangers deleted the hotfix/magento-2-compatibility branch November 14, 2018 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working verified Verified reported behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants