Larastan: NunoMaduro\Larastan\Methods\Macro + bootstrap deprecation

[repat]

• Larastan Version: 7
• --level used:

Since the update to PhpStan 0.12.27

Description

$ vendor/bin/phpstan analyse app
Note: Using configuration file /[...]/phpstan.neon.
⚠️  You're using a deprecated config option bootstrap. ⚠️️

This option has been replaced with bootstrapFiles which accepts a list of files
to execute before the analysis.
[...]
Class NunoMaduro\Larastan\Methods\Macro contains 1 abstract method and must therefore be declared abstract or implement the remaining methods

[szepeviktor]

Yes, this is a known issue.
Please find the answer over at PHPStan https://github.com/phpstan/phpstan/issues/3410#issuecomment-640543460

[szepeviktor]

@repat No! :)
We've fixed it: https://github.com/nunomaduro/larastan/blob/44535adce4fda0ac4d9fe175318fb91959dbb2b4/extension.neon#L34
The next release will come very soon.

Please try installing dev-master

[szepeviktor]

I'm sorry, this is early morning here :)

[szepeviktor]

This is interesting, we do not have abstract methods in Macro
https://github.com/nunomaduro/larastan/blob/v0.5.8/src/Methods/Macro.php

Related #298

Are you on v0.5.8?

[nunomaduro]

@repat Can you try to understand the issue and do a pull request on this? Thanks

[alexey-m-ukolov]

I believe the issue with Macro caused by the new method in the BuiltinMethodReflection interface: https://github.com/phpstan/phpstan-src/commit/79efcd7131e414095360f5ee7947d5ce858baf84#diff-b82d3e974bb18e9a251dc655fbb56adcR12

[canvural]

Both issues are fixed in master. There will be a new release soon :+1: