Skip to content

feat(ci delete): add dry-run flag to ci delete

Sebastian Krebs requested to merge gitlab-community/cli:feature/add-ci-delete-dry-run into main

Description

Added --dry-run to ci-delete

Also refactored ci delete.

  • Separated the actual deletion from the both loops. The filter-loop and the split-args0-loop now only collect the IDs.
  • Removed additional output Deleting pipeline #123456. This only appeared when not using the status filter. Now both behaves identical
  • Removed the usage of utils.StringToInt(i2). This function returned 0 on error, which was passed directly to the API and lead to the misleading error "401 unauthorized".

How has this been tested?

  • go test ./... locally
  • Manually

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)
  • Test gap

Merge request reports