Skip to content

Conversation

@mat3e
Copy link
Contributor

@mat3e mat3e commented Sep 13, 2024

Check List:

Following the contributing guidelines will make it easier for us to review and accept your PR.

@joel-costigliola
Copy link
Member

@mat3e there are build failure because the code needs to be java 8 compatible.

@mat3e mat3e force-pushed the 3.x branch 7 times, most recently from e846bac to 8a37259 Compare January 3, 2025 21:47
@mat3e mat3e marked this pull request as ready for review January 3, 2025 21:49
@mat3e
Copy link
Contributor Author

mat3e commented Jan 3, 2025

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 4.0.0-M1, so shouldn't be a problem 👍🏻 alternatively, for 3.x it could be introduced just explicitly with thenHashSet and assertThatHashSet methods, with then and assertThat since 4 🤔

@mat3e
Copy link
Contributor Author

mat3e commented Jan 3, 2025

I haven't yet updated InstanceOfAssertFactories, BDDAssertions, etc. Let me know if the rest looks conceptually OK.

@scordio scordio changed the base branch from 3.x to main January 4, 2025 10:52
@scordio scordio added this to the 4.0.0-M1 milestone Jan 4, 2025
@scordio scordio modified the milestones: 4.0.0-M1, 4.0.0-M2 Jan 31, 2025
@scordio scordio force-pushed the main branch 2 times, most recently from b6e46b9 to 3f3b814 Compare April 24, 2025 21:24
@joel-costigliola
Copy link
Member

Integrated thanks @mat3e !

@mat3e
Copy link
Contributor Author

mat3e commented Aug 17, 2025

Integrated thanks @mat3e !

Wow, thanks!

I wanted to fix remaining classes during next longer time off from ordinary work, but glad you did it 😇

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.

contains for hashset finds Object which is changed

3 participants