-
Notifications
You must be signed in to change notification settings - Fork 551
Closed
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed
Milestone
Description
Component
JKube Kit
Task description
Description
This loop in ImageStreamService needs some minor refactoring. It uses multiple continue
and even go-to statement. Perhaps it's a remnant of legacy code that was ported from Fabric8 Maven Plugin.
Refactor this loop not to use continue
statements.
Expected Behavior
Loop should not contain any continue
statements
Acceptance Criteria
- Get rid of
continue
statements on line 190, 194, 198 - Get rid of go to statement in nested loop on line 208
- Move nested loop to a separate private method
- ImageStreamServiceTest should pass even after refactoring code
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed