Skip to content

[Php80] Narrow bool|false to bool on UnionTypesRector#3468

Merged
TomasVotruba merged 3 commits intomainfrom
narrow-bool-union
Mar 9, 2023
Merged

[Php80] Narrow bool|false to bool on UnionTypesRector#3468
TomasVotruba merged 3 commits intomainfrom
narrow-bool-union

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@samsonasik samsonasik commented Mar 9, 2023

@samsonasik samsonasik requested a review from TomasVotruba as a code owner March 9, 2023 12:38
@samsonasik
Copy link
Copy Markdown
Member Author

Fixed 🎉 /cc @eugeniya-v

@samsonasik
Copy link
Copy Markdown
Member Author

It somehow cause error on DowngradeMatchToSwitchRectorTest

https://github.com/rectorphp/rector-src/actions/runs/4374536269/jobs/7654085435

I will check more.

@samsonasik
Copy link
Copy Markdown
Member Author

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

@TomasVotruba TomasVotruba merged commit d272dc3 into main Mar 9, 2023
@TomasVotruba TomasVotruba deleted the narrow-bool-union branch March 9, 2023 14:02
@TomasVotruba
Copy link
Copy Markdown
Member

Thank you 👍

@samsonasik
Copy link
Copy Markdown
Member Author

It somehow cause error on downgrade on tracy:

https://github.com/rectorphp/rector-src/actions/runs/4375224842/jobs/7655667800#step:15:70

I will check it.

@samsonasik
Copy link
Copy Markdown
Member Author

#3469

samsonasik added a commit that referenced this pull request May 8, 2023
* [Php80] Narrow bool|false to bool on UnionTypesRector

* Fix phpstan

* try replace union type
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.

UnionTypesRector creates bool|false

2 participants