-
-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Per PR #25 review feedback, doCascadeWithState and doSubmitPushAndPR have accumulated multiple boolean parameters that are easy to misuse at call sites.
Suggested refactoring:
- Create
CascadeOptionsstruct withUpdateOnly,Web,PushOnly,AllBranches,StashReffields - Create
SubmitOptionsstruct withDryRun,UpdateOnly,OpenWeb,PushOnlyfields - Update function signatures and all call sites
This makes calls self-documenting and prevents future flag additions from requiring churn across the codebase.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels