Skip to content

Conversation

@strangelookingnerd
Copy link

@strangelookingnerd strangelookingnerd commented Jan 3, 2025

Fixes regression introduced in 20240325.1 - see #336 for details.

  • Use LinkedHashSet to preserve insertion order where applicable - please note that there may be more regressions like this that have not yet been uncovered
  • Added test that demonstrates the fix works as intended. Without the fix, the test would fail on JDK > 8

@strangelookingnerd
Copy link
Author

Ping @mikesamuel can this please be merged and released?

@strangelookingnerd
Copy link
Author

@jmanico @mikesamuel Kindly requesting a review.

@strangelookingnerd
Copy link
Author

@melloware @aalmiray @jmanico Sorry for the broad ping but I just noticed that the project got some activity again and kindly wanted to raise attention for my regression fix proposed here.

@melloware
Copy link
Contributor

I have no control but I would love to see some of these PRs merged as well

@aalmiray aalmiray force-pushed the main branch 5 times, most recently from 9bb458a to f40152f Compare January 3, 2026 02:00
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.

3 participants