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

TASK: Update Rector #56

Closed
wants to merge 2 commits into from
Closed

TASK: Update Rector #56

wants to merge 2 commits into from

Conversation

mhsdesign
Copy link
Member

currently ci fails after updating phpstan 95355d1 this was reverted again but this revert causes the dev collection not being able to install the newest phpstan 8aa3119.

locally i get:

 rector git:(main) ../../../../bin/phpunit tests --stop-on-failure
PHPUnit 9.6.16 by Sebastian Bergmann and contributors.

E

Time: 00:00.245, Memory: 50.50 MB

There was 1 error:

1) Neos\Rector\Tests\ContentRepository90\Rules\ContextGetFirstLevelNodeCacheRector\ContextGetCurrentRenderingModeRectorTest::test with data set #0 ('/Users/marchenryschultz/Code/...hp.inc')
_PHPStan_7961f7ae1\Nette\DI\ServiceCreationException: Service of type PHPStan\PhpDocParser\Parser\TypeParser: Unable to pass specified arguments to TypeParser::__construct().

phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/Resolver.php:454
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/Resolver.php:174
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/Definitions/ServiceDefinition.php:135
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/Resolver.php:127
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/ContainerBuilder.php:256
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/Compiler.php:203
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/Compiler.php:159
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/ContainerLoader.php:99
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/ContainerLoader.php:65
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/di/src/DI/ContainerLoader.php:34
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/src/DependencyInjection/Configurator.php:56
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/vendor/nette/bootstrap/src/Bootstrap/Configurator.php:177
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/src/DependencyInjection/Configurator.php:68
phar:///Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/phpstan/phpstan/phpstan.phar/src/DependencyInjection/ContainerFactory.php:122
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/packages/NodeTypeResolver/DependencyInjection/PHPStanServicesFactory.php:58
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:915
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:490
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:1023
...
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:490
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:1023
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:976
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:880
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:490
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:1023
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:976
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:880
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:490
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:1023
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:976
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:880
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:490
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:1023
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:976
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:880
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:490
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/vendor/symfony/dependency-injection/ContainerBuilder.php:441
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/packages/Testing/PHPUnit/AbstractTestCase.php:53
/Users/marchenryschultz/Code/core/neos-manufacture-90/Packages/Libraries/rector/rector/packages/Testing/PHPUnit/AbstractRectorTestCase.php:61

ERRORS!
Tests: 1, Assertions: 0, Errors: 1.

Theses issues mention a similar problem and recommend updating rector

rectorphp/rector#7226
rectorphp/rector#7888

Also we dont use phpstan in CI so i removed it.

@mhsdesign
Copy link
Member Author

Also why is it composer run test in ci instead of

"tests": "phpunit tests"
tests??? that cant work lol :D

@mhsdesign
Copy link
Member Author

Moving from 0.15.x to 0.19.x is not easy ... while the releases dont declare any big breaking changes nothing works after an update... and then there is already a stable rector v1 out there ready to be used :O

@kdambekalns
Copy link
Member

Should this be closed in favor of #59?

@mhsdesign
Copy link
Member Author

yes can be, i already had a lot of local things changed but discarded them when i hit the wall

@mhsdesign mhsdesign closed this May 14, 2024
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.

None yet

2 participants