apollo_propeller: rename DuplicateShard/ReceivedSelfPublishedShard variants#13525
Conversation
dee5dc1 to
8e3aabe
Compare
c593904 to
b1c34d2
Compare
8e3aabe to
dc80eaa
Compare
23b47eb to
fba459f
Compare
dc80eaa to
6e1686b
Compare
guy-starkware
left a comment
There was a problem hiding this comment.
@guy-starkware reviewed 4 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on noamsp-starkware and ShahakShama).
6e1686b to
decd756
Compare
06592ea to
3ff06da
Compare
decd756 to
37d2616
Compare
37d2616 to
25a3bc0
Compare
3ff06da to
3704d5e
Compare
PR SummaryLow Risk Overview Updates origin validation ( Reviewed by Cursor Bugbot for commit 2ecd0b8. Bugbot is set up for automated code reviews on this repo. Configure here. |
3704d5e to
977ce48
Compare
25a3bc0 to
2ecd0b8
Compare
Note
Low Risk
Low risk: this is a pure rename of
UnitValidationErrorvariants (and associated call sites/tests) with no behavior changes, but it may break downstream pattern matches if any external crates rely on the old names.Overview
Renames
UnitValidationErrorvariants to use unit terminology (ReceivedSelfPublishedShard->ReceivedSelfPublishedUnit,DuplicateShard->DuplicateUnit) and updates all internal references inPropellerScheduleManager::validate_origin,UnitValidator::validate_unit, and the unit validator tests to match.Reviewed by Cursor Bugbot for commit 37d2616. Bugbot is set up for automated code reviews on this repo. Configure here.