Skip to content

Bko on kiz vmp message size check#8458

Merged
kianenigma merged 4 commits intokiz-vmp-message-size-checkfrom
bko-on-kiz-vmp-message-size-check
May 7, 2025
Merged

Bko on kiz vmp message size check#8458
kianenigma merged 4 commits intokiz-vmp-message-size-checkfrom
bko-on-kiz-vmp-message-size-check

Conversation

@bkontur
Copy link
Copy Markdown
Contributor

@bkontur bkontur commented May 7, 2025

No description provided.

@paritytech-workflow-stopper
Copy link
Copy Markdown

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/14881026966
Failed job name: fmt

@bkontur
Copy link
Copy Markdown
Contributor Author

bkontur commented May 7, 2025

/cmd fmt

/// Pre-check the given UMP message.
fn can_send_upward_message(message: &UpwardMessage) -> Result<(), MessageSendError>;

/// Ensure `[Self::send_upward_message]` is successful when called in benchmarks/tests.
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.

Suggested change
/// Ensure `[Self::send_upward_message]` is successful when called in benchmarks/tests.
/// Ensure [`Self::send_upward_message`] is successful when called in benchmarks/tests.

Copy link
Copy Markdown
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

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

Nice, thanks for the branch! I will merge this and do the final touches in the base PR 🚀

@kianenigma kianenigma merged commit dfe5fe5 into kiz-vmp-message-size-check May 7, 2025
238 of 251 checks passed
@kianenigma kianenigma deleted the bko-on-kiz-vmp-message-size-check branch May 7, 2025 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants