-
-
Notifications
You must be signed in to change notification settings - Fork 761
feat: support HashSet#contains logic for changed hashCodes #3607
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
Conversation
e4fc1ab to
eb7b3ba
Compare
|
@mat3e there are build failure because the code needs to be java 8 compatible. |
assertj-core/src/test/java/org/assertj/core/api/hashset/HashSetAssert_contains_Test.java
Outdated
Show resolved
Hide resolved
assertj-core/src/test/java/org/assertj/core/api/hashset/HashSetAssert_contains_Test.java
Outdated
Show resolved
Hide resolved
e846bac to
8a37259
Compare
|
I allowed myself to resolve the conversations. Took me longer than I thought, but should be good now. It's probably a breaking change, but #1104 is now assigned to |
|
I haven't yet updated |
b6e46b9 to
3f3b814
Compare
|
Integrated thanks @mat3e ! |
Wow, thanks! I wanted to fix remaining classes during next longer time off from ordinary work, but glad you did it 😇 |
Check List:
Following the contributing guidelines will make it easier for us to review and accept your PR.