Skip to content

Fix: Remove empty string elements from IDs slice in delete commands#1451

Merged
ramya18101 merged 2 commits intomainfrom
DXCDT-1468-fix-ids-slice-initialization-empty-strings
Feb 26, 2026
Merged

Fix: Remove empty string elements from IDs slice in delete commands#1451
ramya18101 merged 2 commits intomainfrom
DXCDT-1468-fix-ids-slice-initialization-empty-strings

Conversation

@ramya18101
Copy link
Contributor

@ramya18101 ramya18101 commented Feb 26, 2026

🔧 Changes

Fixes a bug where delete commands across multiple resources were initializing IDs slice with empty strings when arguments were provided via CLI.

The pattern used in delete commands was:

ids := make([]string, len(args))  // Creates slice with len(args) empty strings
// ...
ids = append(ids, args...)         // Appends args after the empty strings

🔬 Testing

  • Manually tested

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@ramya18101 ramya18101 marked this pull request as ready for review February 26, 2026 08:35
@ramya18101 ramya18101 requested a review from a team as a code owner February 26, 2026 08:35
@ramya18101 ramya18101 changed the title fix: update ID slice initialization to handle empty strings Fix: Remove empty string elements from IDs slice in delete commands Feb 26, 2026
@ramya18101 ramya18101 merged commit b31e8f2 into main Feb 26, 2026
6 checks passed
@ramya18101 ramya18101 deleted the DXCDT-1468-fix-ids-slice-initialization-empty-strings branch February 26, 2026 09:07
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.

2 participants