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.

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

ArrayItemRemoval should not mutate lists #1358

Merged

Conversation

sanmai
Copy link
Member

@sanmai sanmai commented Oct 21, 2020

This PR:

Copy link
Member

@maks-rafalko maks-rafalko left a comment

Choose a reason for hiding this comment

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

Reminder for myself: port it to 0.18

src/Mutator/Removal/ArrayItemRemoval.php Outdated Show resolved Hide resolved
@sanmai
Copy link
Member Author

sanmai commented Oct 21, 2020

@maks-rafalko Can we disable this "branch must be up to date" requirement, what do you think? I had to restart the build two times already...

@maks-rafalko
Copy link
Member

@maks-rafalko Can we disable this "branch must be up to date" requirement, what do you think? I had to restart the build two times already...

ok, disabled. This will potentially increase the number of failed builds in master, but let's see

@maks-rafalko maks-rafalko merged commit b8bbd8b into infection:master Oct 21, 2020
@maks-rafalko
Copy link
Member

Thank you @sanmai

maks-rafalko pushed a commit that referenced this pull request Oct 21, 2020
* ArrayItemRemoval should not mutate lists

* Update src/Mutator/Removal/ArrayItemRemoval.php

* Update src/Mutator/Removal/ArrayItemRemoval.php
@maks-rafalko
Copy link
Member

Ported to 0.18, released as 0.18.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

In Assert.php line 2042: Expected a value other than null.
2 participants