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

Add min1 and max1 on NonEmpty for Iterable #14016

Merged
merged 3 commits into from
May 31, 2022
Merged

Add min1 and max1 on NonEmpty for Iterable #14016

merged 3 commits into from
May 31, 2022

Conversation

rgugliel-da
Copy link
Contributor

No description provided.

@rgugliel-da rgugliel-da requested a review from S11001001 May 30, 2022 15:18
@S11001001 S11001001 added the team/ledger-clients Related to the Ledger Clients team's components. label May 31, 2022
S11001001
S11001001 approved these changes May 31, 2022
View reviewed changes
Copy link
Contributor

@S11001001 S11001001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good extension ideas, thanks.

These are in the same category as head1 and tail1; I've added the 1 suffix to them for the same reason it exists on the prior functions.

@S11001001 S11001001 changed the title Add min and max on NonEmtpy for Iterable Add min and max on NonEmpty for Iterable May 31, 2022
@S11001001
no changelog
b101c83 
CHANGELOG_BEGIN
CHANGELOG_END
@S11001001 S11001001 enabled auto-merge (squash) May 31, 2022 12:38
@S11001001 S11001001 changed the title Add min and max on NonEmpty for Iterable Add min1 and max1 on NonEmpty for Iterable May 31, 2022
@S11001001 S11001001 merged commit 291bb5d into main May 31, 2022
@S11001001 S11001001 deleted the reguglien-nonempty-min-max branch May 31, 2022 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@S11001001 S11001001

Assignees

@S11001001 S11001001

@rgugliel-da rgugliel-da

Labels
team/ledger-clients Related to the Ledger Clients team's components.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rgugliel-da @S11001001