Redesign to remove PipelineResult.NotRun
#2443
Labels
Powderhouse
Work to isolate parser and features
PipelineResult.NotRun
#2443
@Keboo correctly questioned the behavior or the
NotRun
method ofPipelineResult
which did nothing.In the current design, this method could be removed (it is currently called from
CliSubstem
) or could reset the default values. Doing nothing is wrong.However, it appears to point to a deeper issue. Multiple subsystems will be called, and we should track these in a thoughtful way.
Initial thoughts:
PipelineResult
needs to be changed. (NoteCheckIfActivated
blocks running unneeded subsystems, so it is a niche case if more than one is needed)If we have exactly one terminating subsystem, we may want the current design plus a list of subsystems that run. If we allow multiple terminating subsystems, then we probably want a list of all subsystems.
The text was updated successfully, but these errors were encountered: