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.

Sign up for GitHub

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

Jump to bottom

Add PregMatchRemoveFlags mutator - remove flags one by one #1462

Merged
merged 1 commit into from Dec 13, 2020
Merged

Add PregMatchRemoveFlags mutator - remove flags one by one #1462

merged 1 commit into from Dec 13, 2020

Conversation

maks-rafalko
Copy link
Member

@maks-rafalko maks-rafalko commented Dec 13, 2020 •
edited

This PR:

- preg_match('/^test$/igu', $string);

+ preg_match('/^test$/gu', $string);
+ preg_match('/^test$/iu', $string);
+ preg_match('/^test$/ig', $string);

@maks-rafalko maks-rafalko added this to the 0.21.0 milestone Dec 13, 2020
sidz
sidz approved these changes Dec 13, 2020
View reviewed changes
Copy link
Member

@sidz sidz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

maks-rafalko added a commit to infection/site that referenced this pull request Dec 13, 2020
@maks-rafalko
Add docs for new regex mutator - PregMatchRemoveFlags
274a114 
related to infection/infection#1462
@maks-rafalko maks-rafalko mentioned this pull request Dec 13, 2020
Merged
@maks-rafalko maks-rafalko merged commit ebc39a2 into master Dec 13, 2020
@maks-rafalko maks-rafalko deleted the feature/preg-match-remove-flags branch December 13, 2020 20:58
@maks-rafalko
Copy link
Member Author

Thank you @sidz

@renovate renovate bot mentioned this pull request Jun 6, 2021
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sidz sidz sidz approved these changes

Assignees
No one assigned
Labels
Mutator
Projects
None yet
Milestone
0.21.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@maks-rafalko @sidz