EasyPost / easypost-java

EasyPost Shipping API Client Library for Java

https://easypost.com/docs/api

MIT License

54 stars 37 forks source link

remove assertj dependency #194

Closed jchen293 closed 2 years ago

jchen293 commented 2 years ago

Description

Remove assertj dependency since it's no longer being used, this will also make a dependency less on the Test Dependencies in our Java Maven page. We have overridden the equal() for object comparison in this PR

Testing

Current unit tests pass.

Pull Request Type

Please select the option(s) that are relevant to this PR.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Improvement (fixing a typo, updating readme, renaming a variable name, etc)

jchen293 commented 2 years ago

Nice work identifying this! This isn't being imported anywhere?

Nope, I have removed them in the override equal() PR