Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Misc fixes #78

Merged
merged 2 commits into from
Jan 22, 2025
Merged

Misc fixes #78

merged 2 commits into from
Jan 22, 2025

Conversation

tginsberg
Copy link
Owner

  • Use LinkedHashSet in UniquelyOccurringGatherer instead of a Map
  • Fix nullability annotation order (makes IntelliJ stop superfluous warnings)

@tginsberg
Misc fixes
31006b0 
+ Alter nullity annotation order
+ Reimplement UniquelyOccurringGatherer to use LinkedHashSet
@tginsberg
Implement Greedy
786aa30
@github-actions GitHub Actions
Copy link

Overall Project 99.11% 🍏
Files changed 100% 🍏

File Coverage
GroupingByGatherer.java 99.04% 🍏
BigDecimalGatherer.java 98.63% 🍏
UniquelyOccurringGatherer.java 98.46% 🍏
DedupeConsecutiveGatherer.java 98.25% 🍏
EveryNthGatherer.java 97.78% 🍏
GathererUtils.java 88.89% 🍏

@tginsberg tginsberg merged commit 588fc50 into release/0.8.0 Jan 22, 2025
2 checks passed
@tginsberg tginsberg deleted the feature/misc-fixes branch January 22, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tginsberg