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

add iter-from for bags-list #11104

Merged
merged 5 commits into from
Mar 31, 2022
Merged

Conversation

kianenigma
Copy link
Contributor

@kianenigma kianenigma commented Mar 23, 2022

Small backport needed in paritytech/polkadot-sdk#461
polkadot companion: paritytech/polkadot#5209

@kianenigma kianenigma added A0-please_review Pull request needs code review. 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. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Mar 23, 2022
frame/staking/src/pallet/impls.rs Show resolved Hide resolved
frame/bags-list/src/tests.rs Show resolved Hide resolved
} else {
Err(())
}
}
Copy link
Member

Choose a reason for hiding this comment

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

Could you maybe also add a minimal test here?

Co-authored-by: Oliver Tale-Yazdi <[email protected]>
Copy link
Contributor

@emostov emostov left a comment

Choose a reason for hiding this comment

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

LGTM

@kianenigma
Copy link
Contributor Author

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Checks failed for c3eb6af

@kianenigma
Copy link
Contributor Author

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit 2287cd8 into master Mar 31, 2022
@paritytech-processbot paritytech-processbot bot deleted the kiz-iter-from-bags-list branch March 31, 2022 14:16
DaviRain-Su pushed a commit to octopus-network/substrate that referenced this pull request Aug 23, 2022
* add iter-from for bags-list

* Fix

* Apply suggestions from code review

Co-authored-by: Oliver Tale-Yazdi <[email protected]>

* Fix

Co-authored-by: Oliver Tale-Yazdi <[email protected]>
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* add iter-from for bags-list

* Fix

* Apply suggestions from code review

Co-authored-by: Oliver Tale-Yazdi <[email protected]>

* Fix

Co-authored-by: Oliver Tale-Yazdi <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. 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. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants