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

feat(Added linq mutator): Added linq Union and intersect mutator. #1182

Merged
merged 2 commits into from
Sep 4, 2020
Merged

Conversation

Myoxocephalus
Copy link
Contributor

Added linq mutators as proposed in #1038

@richardwerkman
Copy link
Member

Thank you for your contribution! I do have one question. Why did you only add these mutations and not all suggested mutations in the issue? If you feel like it you could add the other two suggested mutations as well.

@Myoxocephalus
Copy link
Contributor Author

Hi!
Thanks for the comment!
I think the other mutations makes sense as well. The reason i scoped this PR was mainly because im a new contributor to this project and wanted to start small and get some feedback.
I could create an additional PR for that other mutation.

@richardwerkman richardwerkman merged commit edf7ebb into stryker-mutator:master Sep 4, 2020
@richardwerkman
Copy link
Member

Thanks for your contribution! If you like to add the other mutations as well, please open another PR 😃

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.

2 participants