Skip to content

Remove pallet-scored-pool - #12746

Open
Ank4n wants to merge 2 commits into
masterfrom
ankan/remove-pallet-scored-pool
Open

Remove pallet-scored-pool#12746
Ank4n wants to merge 2 commits into
masterfrom
ankan/remove-pallet-scored-pool

Conversation

@Ank4n

@Ank4n Ank4n commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes pallet-scored-pool.

Rationale, as raised in #12333 (comment):

  • Unused. No runtime in this repository (relay chains, system parachains, templates, kitchensink) includes it. Its only in-tree consumer was its own mock.
  • Unmaintained. No functional change in years — the last several commits touching it were purely mechanical (rustfmt rule rollouts, RuntimeEvent removal, dep cleanup, Rust version bumps).
  • Superseded in practice. Membership/ranking use cases in the SDK are served by pallet-membership, pallet-ranked-collective, and pallet-core-fellowship.
  • Not staking-related. As confirmed on [RWF] pallet-scored-pool: fix pool sort logic #12333, staking does not interact with this pallet, so there is no impact on the current or the async staking system.

This supersedes #12333, which fixes a sort-order bug in the pallet being removed here.

Changes

  • Delete substrate/frame/scored-pool/
  • Drop the workspace member entry
  • Drop the pallet-scored-pool optional dependency, feature, std/try-runtime propagation, and re-export from the polkadot-sdk umbrella crate

Notes for reviewers

Per the deprecation checklist, the usual path is a hard-deprecation notice with a ~6 month removal window before removal. I've gone straight to removal here since the pallet is unused in-tree and has no known production users, but I'm happy to split this into a deprecate-then-remove sequence instead if reviewers prefer to follow the checklist strictly — the crate is published on crates.io, so external builders could in principle be depending on it.

pallet-scored-pool is intentionally not listed in the prdoc crates section: check-prdoc.py validates crate names against the workspace, so a deleted crate cannot be listed there.

🤖 Generated with Claude Code

This pallet is not used by any runtime in the repository, has had no
functional changes in years (only mechanical maintenance touches), and
has no known production users.

Removes the crate, its workspace membership, and its umbrella crate
re-export and features.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@Ank4n
Ank4n requested a review from a team as a code owner July 28, 2026 13:23
@cla-bot-2021

cla-bot-2021 Bot commented Jul 28, 2026

Copy link
Copy Markdown

User @claude, please sign the CLA here.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@Ank4n Ank4n added T2-pallets This PR/Issue is related to a particular pallet. T13-deprecation The current issue/pr is, or should be, part of a deprecation process. labels Jul 28, 2026

@sigurpol sigurpol left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

RIP. I am fine with your approach of fast-tracking the removal. I would also be perfectly fine with the default deprecation checklist path if we want to play safer just in case.

@Ank4n

Ank4n commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

Nice work Claude!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T2-pallets This PR/Issue is related to a particular pallet. T13-deprecation The current issue/pr is, or should be, part of a deprecation process.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants