-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Override hashCode in Quantity #3495
Override hashCode in Quantity #3495
Conversation
|
Welcome @nulls! |
3edf927
to
ed0fa80
Compare
/lgtm Thanks for the PR! |
ed0fa80
to
901ea58
Compare
hi, thanks for the review. I amended the commit message to fix it. Can you please re-appove? |
- overrode `hashCode` in `Quantity` - added test which compares two sets of equal elements
901ea58
to
62d3dad
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: brendandburns, nulls, orchestr7 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
What's done:
hashCode
inQuantity
It
closes
#3490