Skip to content

Do not support array of patterns in Preg methods#6184

Merged
SpacePossum merged 1 commit intoPHP-CS-Fixer:masterfrom
werlos:do_not_support_arrays_in_Preg_methods
Dec 16, 2021
Merged

Do not support array of patterns in Preg methods#6184
SpacePossum merged 1 commit intoPHP-CS-Fixer:masterfrom
werlos:do_not_support_arrays_in_Preg_methods

Conversation

@kubawerlos
Copy link
Member

@kubawerlos kubawerlos commented Dec 15, 2021

Requires #6183 to be merged.

Preg is an internal class, so BC break is fine.

To be continued: #6185.

This was referenced Dec 15, 2021
SpacePossum added a commit that referenced this pull request Dec 16, 2021
This PR was merged into the master branch.

Discussion
----------

Simplify DiffConsoleFormatter

This is actually only place where any `Preg` method is called with an array (of patterns), not a string.

To be continued: #6184.

Commits
-------

cf4dcc4 Simplify DiffConsoleFormatter
@SpacePossum
Copy link
Contributor

please rebase :)

@kubawerlos kubawerlos marked this pull request as ready for review December 16, 2021 08:24
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 93.037% when pulling b610b92 on werlos:do_not_support_arrays_in_Preg_methods into b4b6d13 on FriendsOfPHP:master.

@SpacePossum
Copy link
Contributor

Thank you @kubawerlos.

@SpacePossum SpacePossum merged commit d23cc90 into PHP-CS-Fixer:master Dec 16, 2021
@kubawerlos kubawerlos deleted the do_not_support_arrays_in_Preg_methods branch December 16, 2021 08:45
SpacePossum added a commit that referenced this pull request Dec 16, 2021
This PR was merged into the master branch.

Discussion
----------

Upgrade PHPStan

Requires #6184 to be merged.

After merging the PR above CI will be green.

The end 🎆

As an extra you can see here PHPStan error annotation in the diff (until it is fixed of course).

Commits
-------

5661979 Upgrade PHPStan
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments