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

BigDecimal mutator #965

Merged
merged 1 commit into from Dec 15, 2021
Merged

BigDecimal mutator #965

merged 1 commit into from Dec 15, 2021

Conversation

drnow4u
Copy link
Contributor

@drnow4u drnow4u commented Dec 13, 2021

Hi @hcoles,

Thanks for outstanding project Pitest. I’m using it for years. Pitest let think outside the box about problem I'm workin on.

I'm often doing calculation based on BigDecimal. My implementation is similar to BigIntegerMutator. I read conversation in #518 (comment), but I couldn't find reason to not implement BigDecimal mutator.

Could you review? Suggestions are always welcome.

Kindly regards,

@hcoles
Copy link
Owner

hcoles commented Dec 15, 2021

@MarcinNowak-codes Ignore my earlier comments, they are incorrect. Merging.

@hcoles hcoles merged commit 1c2d301 into hcoles:master Dec 15, 2021
@drnow4u
Copy link
Contributor Author

drnow4u commented Dec 15, 2021

Thank you @hcoles

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.

None yet

2 participants