Skip to content

apollo_propeller: fix PropellerScheduleManager doc comments to say 'units'#13530

Open
sirandreww-starkware wants to merge 1 commit into03-26-apollo_propeller_remove_incorrect_tree.rs_module_doc_commentfrom
03-26-apollo_propeller_fix_propellerschedulemanager_doc_comments_to_say_units_
Open

apollo_propeller: fix PropellerScheduleManager doc comments to say 'units'#13530
sirandreww-starkware wants to merge 1 commit into03-26-apollo_propeller_remove_incorrect_tree.rs_module_doc_commentfrom
03-26-apollo_propeller_fix_propellerschedulemanager_doc_comments_to_say_units_

Conversation

@sirandreww-starkware
Copy link
Copy Markdown
Contributor

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

Note

Low Risk
Doc-only change that updates protocol terminology from "shards" to "units"; no runtime behavior is modified.

Overview
Updates PropellerScheduleManager module-level documentation in tree.rs to consistently refer to received/broadcast units (instead of shards) when describing the build/receive thresholds and broadcast behavior.

Written by Cursor Bugbot for commit bc9519d. 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 16:01
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_propellerschedulemanager_doc_comments_to_say_units_ branch from 1bf590f to bc9519d 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