feat(local exec): add ability to skip steps

[KellyMerrick]

Added flags for skipping steps:

cli/command/pipeline/exec.go

Lines 137 to 144 in a51e7aa

	// Step Flags
	&cli.StringSliceFlag{
	EnvVars: []string{"VELA_SKIP_STEP", "SKIP_STEP"},
	Name: "skip-step",
	Aliases: []string{"sk", "skip"},
	Usage: "skip a step in the pipeline",
	},

Works for steps, stages, anchors, yaml templates, nested yaml templates.

• Stages: any steps with the provided name will be skipped, across all stages.
• Templates and nested templates: prepend the template name(s) to the step name
• Step names with spaces: wrap in quotes, including template name(s).

$ vela exec pipeline --sk echo_hi --sk child_echo_hi --sk 'child_some grandchild_echo_hi'

cli/action/pipeline/exec.go

Lines 118 to 171 in a51e7aa

	// create a slice for steps to be removed
	stepsToRemove := c.Steps
	// print steps to be removed to the user
	if len(stepsToRemove) > 0 {
	for _, stepName := range stepsToRemove {
	fmt.Println("skip step: ", stepName)
	}
	}
	// filter out steps to be removed
	if len(_pipeline.Stages) > 0 {
	// if using stages
	// counter for total steps to run
	totalSteps := 0
	for i, stage := range _pipeline.Stages {
	filteredStageSteps := stage.Steps[:0]
	for _, step := range stage.Steps {
	// if c.steps contains step.Name
	if !slices.Contains(stepsToRemove, step.Name) {
	filteredStageSteps = append(filteredStageSteps, step)
	totalSteps++
	}
	_pipeline.Stages[i].Steps = filteredStageSteps
	}
	}
	// check if any steps are left to run, excluding "init" step
	if totalSteps <= 1 {
	return fmt.Errorf("no steps left to run after removing skipped steps")
	}
	} else {
	// if not using stages
	filteredSteps := _pipeline.Steps[:0]
	for _, step := range _pipeline.Steps {
	if !slices.Contains(stepsToRemove, step.Name) {
	filteredSteps = append(filteredSteps, step)
	}
	}
	_pipeline.Steps = filteredSteps
	// check if any steps are left to run, excluding "init" step
	if len(_pipeline.Steps) <= 1 {
	return fmt.Errorf("no steps left to run after removing skipped steps")
	}
	}

Also corrected punctuation for proper display in command help output:

cli/command/pipeline/exec.go

Line 163 in a51e7aa

Aliases: []string{"tf", "tfs", "template-files"},

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 40 lines in your changes missing coverage. Please review.

Project coverage is 78.97%. Comparing base (55c73d4) to head (7d8f639).

Files with missing lines	Patch %	Lines
action/pipeline/exec.go	0.00%	39 Missing ⚠️
command/pipeline/exec.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #584      +/-   ##
==========================================
- Coverage   79.38%   78.97%   -0.41%     
==========================================
  Files         172      172              
  Lines        7728     7768      +40     
==========================================
  Hits         6135     6135              
- Misses       1357     1397      +40     
  Partials      236      236              

Files with missing lines	Coverage Δ
command/pipeline/exec.go	0.00% <0.00%> (ø)
action/pipeline/exec.go	0.00% <0.00%> (ø)

[ecrupper]

Suggested change

	Steps []string
	SkipSteps []string

Thoughts on making this field more descriptive?

[ecrupper]

Thoughts on moving this inside the loops below as we prune the pipeline?