Skip to content

apollo_propeller: fix UnitValidationError error messages to say 'unit' instead of 'shard'#13524

Open
sirandreww-starkware wants to merge 1 commit into03-26-apollo_propeller_fix_unitvalidationerror_doc_comment_to_say_unit_verification_from
03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_
Open

apollo_propeller: fix UnitValidationError error messages to say 'unit' instead of 'shard'#13524
sirandreww-starkware wants to merge 1 commit into03-26-apollo_propeller_fix_unitvalidationerror_doc_comment_to_say_unit_verification_from
03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_

Conversation

@sirandreww-starkware
Copy link
Copy Markdown
Contributor

@sirandreww-starkware sirandreww-starkware commented Mar 26, 2026

Note

Low Risk
Low risk: string-only changes to error messages with no behavior or data-flow impact.

Overview
Updates UnitValidationError’s thiserror messages in types.rs to consistently use unit terminology (instead of shard) across self-send, duplicate, schedule/parent verification, signature, and Merkle proof failure cases.

Written by Cursor Bugbot for commit c19fe7c. This will update automatically on new commits. Configure here.

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@sirandreww-starkware sirandreww-starkware self-assigned this Mar 26, 2026
@sirandreww-starkware sirandreww-starkware marked this pull request as ready for review March 26, 2026 15:59
Copy link
Copy Markdown
Contributor Author

sirandreww-starkware commented Mar 26, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_ branch from 3aca3da to c19fe7c Compare March 26, 2026 18:18
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