search

pitest / pitclipse

Mutation testing for Java in Eclipse IDE. Based on PIT (Pitest).
https://pitest.org
Apache License 2.0
59 stars 17 forks source link

Refactoring #132

Closed LorenzoBettini closed 3 years ago

LorenzoBettini commented 3 years ago

PitclipseService was used only in its own PitclipseServiceTest so I removed them both

LorenzoBettini commented 3 years ago

@echebbi what do you think?

LorenzoBettini commented 3 years ago

@echebbi I'd like to merge it is that OK?

echebbi commented 3 years ago

@LorenzoBettini if the files aren't actually used, why did you moved them instead of merely removed them?

LorenzoBettini commented 3 years ago

@LorenzoBettini if the files aren't actually used, why did you moved them instead of merely removed them?

I just wanted to be sure ;) if you prefer I'll remove them directly

sonarcloud[bot] commented 3 years ago

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

echebbi commented 3 years ago

Yes, I think it's cleaner that way. If those files ever end up appearing useful we'll still be able to recover them with Git.