Skip to content

try state hook for pallet authorship#11215

Merged
bkchr merged 6 commits intoparitytech:masterfrom
dharjeezy:dami/pallet-authorship-try-state-check
Mar 23, 2026
Merged

try state hook for pallet authorship#11215
bkchr merged 6 commits intoparitytech:masterfrom
dharjeezy:dami/pallet-authorship-try-state-check

Conversation

@dharjeezy
Copy link
Copy Markdown
Contributor

This PR introduces the try_state hook to pallet-authorship to verify a key storage invariant.

closes part of #239

@dharjeezy dharjeezy requested a review from a team as a code owner March 1, 2026 16:47
@dharjeezy
Copy link
Copy Markdown
Contributor Author

/cmd prdoc

@dharjeezy
Copy link
Copy Markdown
Contributor Author

/cmd label T2-pallets

@paritytech-cmd-bot-polkadot-sdk paritytech-cmd-bot-polkadot-sdk Bot added the T2-pallets This PR/Issue is related to a particular pallet. label Mar 1, 2026
Comment thread prdoc/pr_10475.prdoc Outdated
This PR introduces the try_state hook to pallet-authorship to verify key storage invariants.
crates:
- name: pallet-authority-discovery
- name: pallet-authorship
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why changing this file?

Comment thread prdoc/pr_11215.prdoc Outdated
@@ -0,0 +1,10 @@
title: try state hook for pallet authorship
doc:
- audience: Todo
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
- audience: Todo
- audience: Runtime Dev

Comment thread prdoc/pr_11215.prdoc Outdated
closes part of https://github.com/paritytech/polkadot-sdk/issues/239
crates:
- name: pallet-authorship
bump: major
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
bump: major
bump: minor

@dharjeezy dharjeezy requested a review from ggwpez March 23, 2026 10:31
@bkchr bkchr enabled auto-merge March 23, 2026 22:42
@bkchr bkchr added this pull request to the merge queue Mar 23, 2026
Merged via the queue into paritytech:master with commit 6155693 Mar 23, 2026
249 of 252 checks passed
clangenb pushed a commit to clangenb/polkadot-sdk that referenced this pull request Mar 24, 2026
This PR introduces the try_state hook to pallet-authorship to verify a
key storage invariant.

closes part of paritytech#239

---------

Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
franciscoaguirre pushed a commit that referenced this pull request Mar 24, 2026
This PR introduces the try_state hook to pallet-authorship to verify a
key storage invariant.

closes part of #239

---------

Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants