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

Rename NposSolver trait to ElectionSolver #429

Open
gpestana opened this issue Feb 15, 2023 · 0 comments
Open

Rename NposSolver trait to ElectionSolver #429

gpestana opened this issue Feb 15, 2023 · 0 comments
Labels
I10-unconfirmed Issue might be valid, but it's not yet known.

Comments

@gpestana
Copy link
Contributor

Once paritytech/substrate#12588 is merged, we may want to consider renaming the election traits and helpers to be more generic, since the election solver implementor may not be npos related.
@juangirini juangirini transferred this issue from paritytech/substrate Aug 24, 2023
@the-right-joyce the-right-joyce added I10-unconfirmed Issue might be valid, but it's not yet known. and removed J2-unconfirmed labels Aug 25, 2023
jonathanudd pushed a commit to jonathanudd/polkadot-sdk that referenced this issue Apr 10, 2024
@svyatonik @HCastano @bkchr
InboundLaneApi::latest_confirmed_nonce (paritytech#429)
5accb2b 
* InboundLaneApi::latest_confirmed_nonce

* Fix wording and typo

Co-authored-by: Hernando Castano <[email protected]>
This was referenced Jun 5, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
I10-unconfirmed Issue might be valid, but it's not yet known.
Projects
(Nominated) Proof of Stake
Status: 📕 Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gpestana @the-right-joyce