Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Fix max > T::MaxCandidates benchmark warning #2716

Merged
merged 1 commit into from
Jun 8, 2023
Merged

Fix max > T::MaxCandidates benchmark warning #2716

merged 1 commit into from
Jun 8, 2023

Conversation

rlaferla
Copy link
Contributor

@rlaferla rlaferla commented Jun 8, 2023

Closes #2715

See also:
#2715

@rlaferla rlaferla mentioned this pull request Jun 8, 2023
Closed
@bkchr bkchr requested a review from ggwpez June 8, 2023 20:42
gilescope
gilescope approved these changes Jun 8, 2023
View reviewed changes
Copy link
Contributor

@gilescope gilescope left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@bkchr bkchr added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Jun 8, 2023
@bkchr bkchr merged commit 766da34 into paritytech:master Jun 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@gilescope gilescope gilescope approved these changes

@bkchr bkchr bkchr approved these changes

@ggwpez ggwpez Awaiting requested review from ggwpez

Assignees
No one assigned
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Milestone
No milestone
3 participants
@rlaferla @gilescope @bkchr