Skip to content

[Php74] Add ParenthesizeNestedTernaryRector#2859

Merged
TomasVotruba merged 12 commits intomainfrom
add-parenthesize-nested-ternary
Aug 31, 2022
Merged

[Php74] Add ParenthesizeNestedTernaryRector#2859
TomasVotruba merged 12 commits intomainfrom
add-parenthesize-nested-ternary

Conversation

@samsonasik
Copy link
Member

@samsonasik
Copy link
Member Author

implemented 🎉 /cc @fruitl00p

@samsonasik samsonasik marked this pull request as draft August 30, 2022 10:25
@samsonasik samsonasik force-pushed the add-parenthesize-nested-ternary branch 2 times, most recently from b8258f1 to 8992f47 Compare August 30, 2022 10:38
@samsonasik samsonasik marked this pull request as ready for review August 30, 2022 10:47
@samsonasik
Copy link
Member Author

All checks have passed 🎉 @TomasVotruba it is ready for review.

@samsonasik samsonasik force-pushed the add-parenthesize-nested-ternary branch from d30f347 to 44f4b8a Compare August 31, 2022 06:07
@samsonasik
Copy link
Member Author

rebased and regenerate docs

@samsonasik
Copy link
Member Author

@TomasVotruba tests error seems related with removal of symplify/easy-testing at PR:

which cause error:

There were 8 errors:

1) Error
The data provider specified for Rector\Tests\Naming\Naming\UseImportsResolver\UseImportsResolverTest::testUsesFromProperty is invalid.
Error: Class "Symplify\EasyTesting\DataProvider\StaticFixtureFinder" not found
/home/runner/work/rector-src/rector-src/rules-tests/Naming/Naming/UseImportsResolver/UseImportsResolverTest.php:64

ref https://github.com/rectorphp/rector-src/runs/8107310730?check_suite_focus=true#step:5:76

@TomasVotruba TomasVotruba merged commit 6db4c0e into main Aug 31, 2022
@TomasVotruba TomasVotruba deleted the add-parenthesize-nested-ternary branch August 31, 2022 07:04
@TomasVotruba
Copy link
Member

Thank you 👍

,I'm merging the PR as tests are in another PR

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.

Nested ternary expressions are depreceted since 7.4

3 participants