feat(ci delete): add dry-run flag to ci delete
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 returned0
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