Skip to content

fix: remove dead duplicate Running status check in start command#290

Open
mynk8 wants to merge 1 commit intomicrocks:masterfrom
mynk8:fix/duplicate-running-status-check
Open

fix: remove dead duplicate Running status check in start command#290
mynk8 wants to merge 1 commit intomicrocks:masterfrom
mynk8:fix/duplicate-running-status-check

Conversation

@mynk8
Copy link
Copy Markdown

@mynk8 mynk8 commented May 5, 2026

The if instance.Status == "Running" check at line 50 already returns early, making the identical case "Running": inside the switch statement unreachable dead code.

@mynk8 mynk8 requested review from Harsh4902, lbroudoux and yada as code owners May 5, 2026 13:31
@mynk8 mynk8 force-pushed the fix/duplicate-running-status-check branch from 37748a0 to d96fe6d Compare May 5, 2026 13:33
The if-block above the switch was redundant with the case inside it.
Consolidate into the switch statement only.

Signed-off-by: mynk8 <mr.mayankgame@gmail.com>
@mynk8 mynk8 force-pushed the fix/duplicate-running-status-check branch from d96fe6d to 6556d46 Compare May 5, 2026 13:35
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.

1 participant