Skip to content

Allow justifications on non-finalized blocks#1211

Merged
bkchr merged 3 commits intoparitytech:masterfrom
nazar-pc:allow-justifications-on-non-finalized-blocks
Sep 12, 2023
Merged

Allow justifications on non-finalized blocks#1211
bkchr merged 3 commits intoparitytech:masterfrom
nazar-pc:allow-justifications-on-non-finalized-blocks

Conversation

@nazar-pc
Copy link
Copy Markdown
Contributor

One assertion was unnecessary because of the check right above it, second assertion resolves #1159

@bkchr bkchr added the T0-node This PR/Issue is related to the topic “node”. label Aug 30, 2023
@bkchr bkchr requested a review from arkpar August 30, 2023 08:56
@nazar-pc
Copy link
Copy Markdown
Contributor Author

Friendly ping here

@bkchr bkchr requested a review from andresilva as a code owner September 12, 2023 14:05
@bkchr bkchr enabled auto-merge (squash) September 12, 2023 14:05
@bkchr bkchr merged commit ee6eeb7 into paritytech:master Sep 12, 2023
@nazar-pc nazar-pc deleted the allow-justifications-on-non-finalized-blocks branch September 15, 2023 13:49
bkchr pushed a commit that referenced this pull request Sep 25, 2023
…ment (#1607)

I was reading source code and noticed these two places that are no
longer true after #1211
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
One assertion was unnecessary because of the check right above it,
second assertion resolves
paritytech#1159

---------

Co-authored-by: Bastian Köcher <git@kchr.de>
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
…ment (paritytech#1607)

I was reading source code and noticed these two places that are no
longer true after paritytech#1211
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T0-node This PR/Issue is related to the topic “node”.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow justifications on non-finalized blocks

4 participants