Skip to content

[CodeQuality] Skip with ascii on JoinStringConcatRector#3497

Merged
samsonasik merged 6 commits intomainfrom
skip-with-multibyte
Mar 21, 2023
Merged

[CodeQuality] Skip with ascii on JoinStringConcatRector#3497
samsonasik merged 6 commits intomainfrom
skip-with-multibyte

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@samsonasik
Copy link
Copy Markdown
Member Author

Fixed 🎉 /cc @vralle

@samsonasik
Copy link
Copy Markdown
Member Author

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

@samsonasik samsonasik changed the title [CodeQuality] Skip with multibyte on JoinStringConcatRector [CodeQuality] Skip with ascii on JoinStringConcatRector Mar 21, 2023
@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba I think it is ready.

@samsonasik
Copy link
Copy Markdown
Member Author

@TomasVotruba I am merging it ;)

@samsonasik samsonasik merged commit 28e9cdb into main Mar 21, 2023
@samsonasik samsonasik deleted the skip-with-multibyte branch March 21, 2023 02:15
samsonasik added a commit that referenced this pull request May 8, 2023
* [CodeQuality] Skip with multibyte on JoinStringConcatRector

* Fixed 🎉

* eol

* verify with regex

* rename fixture

* fix phpstan
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.

Unexpected behavior with JoinStringConcatRector rule

1 participant