Skip to content

remove legacy durable pipeline cmd#875

Merged
ajaskolski merged 1 commit intomainfrom
chore-remove-legacy-durable-pipeline-cmd
Mar 19, 2026
Merged

remove legacy durable pipeline cmd#875
ajaskolski merged 1 commit intomainfrom
chore-remove-legacy-durable-pipeline-cmd

Conversation

@ajaskolski
Copy link
Copy Markdown
Contributor

@ajaskolski ajaskolski commented Mar 19, 2026

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 19, 2026

⚠️ No Changeset found

Latest commit: 459bb73

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@cl-sonarqube-production
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@ajaskolski ajaskolski marked this pull request as ready for review March 19, 2026 12:45
@ajaskolski ajaskolski requested a review from a team as a code owner March 19, 2026 12:45
Copilot AI review requested due to automatic review settings March 19, 2026 12:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the legacy durable pipeline CLI command implementation and its associated helper/template logic from the engine/cld/legacy/cli/commands package, aligning with CLD-1247’s goal of clearing unused pipeline code (with the modern engine/cld/commands/pipeline command already providing the durable-pipeline alias).

Changes:

  • Deleted the legacy durable-pipeline command implementation (run/input-generate/list/template-input) from the legacy CLI commands package.
  • Removed legacy YAML parsing/input env-var helpers tied to durable pipeline execution.
  • Removed the legacy test suites that exercised the deleted command and template generation logic.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
engine/cld/legacy/cli/commands/template_input_test.go Removed tests for legacy template-input YAML generation.
engine/cld/legacy/cli/commands/template_input.go Removed legacy template-input YAML generation implementation.
engine/cld/legacy/cli/commands/durable_pipeline_yaml.go Removed legacy YAML parsing + env-var setting utilities for durable pipeline inputs.
engine/cld/legacy/cli/commands/durable-pipelines_test.go Removed legacy durable-pipelines command tests (run/input-generate/list/etc.).
engine/cld/legacy/cli/commands/durable-pipelines_helper_test.go Removed tests for legacy changeset config resolution helpers.
engine/cld/legacy/cli/commands/durable-pipelines_helper.go Removed legacy helpers (workspace root, YAML path resolution, config resolution, env option config).
engine/cld/legacy/cli/commands/durable-pipelines.go Removed legacy durable-pipeline command implementation and subcommands.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ajaskolski ajaskolski added this pull request to the merge queue Mar 19, 2026
Merged via the queue into main with commit ca220b0 Mar 19, 2026
25 checks passed
@ajaskolski ajaskolski deleted the chore-remove-legacy-durable-pipeline-cmd branch March 19, 2026 13:23
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.

3 participants