Skip to content

fix: added validation for --skip-consumers with default lookup consum…#1875

Merged
shivaygupta-dotcom merged 4 commits intomainfrom
fix/FTI-7230
Feb 6, 2026
Merged

fix: added validation for --skip-consumers with default lookup consum…#1875
shivaygupta-dotcom merged 4 commits intomainfrom
fix/FTI-7230

Conversation

@shivaygupta-dotcom
Copy link
Contributor

@shivaygupta-dotcom shivaygupta-dotcom commented Feb 4, 2026

#1844
https://konghq.atlassian.net/browse/FTI-7230

We have added a validation on deck, where --skip-consumers flag with default-look-up tags(consumer and consumer-groups) shouldn't be allowed together.

@codecov-commenter
Copy link

codecov-commenter commented Feb 4, 2026

Codecov Report

❌ Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 32.74%. Comparing base (e4c6528) to head (fcc5e0b).

Files with missing lines Patch % Lines
cmd/common.go 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1875      +/-   ##
==========================================
- Coverage   32.78%   32.74%   -0.05%     
==========================================
  Files          73       73              
  Lines        6557     6566       +9     
==========================================
  Hits         2150     2150              
- Misses       4243     4252       +9     
  Partials      164      164              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@shivaygupta-dotcom shivaygupta-dotcom merged commit 9f67d92 into main Feb 6, 2026
39 of 41 checks passed
@shivaygupta-dotcom shivaygupta-dotcom deleted the fix/FTI-7230 branch February 6, 2026 07:42
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