litmuschaos/litmus

Chore: remove inefficient code

Aperta

#4206 aperta il 4 ott 2023

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (1 assegnatario)HTML (574 fork)batch import
Hacktoberfestgood first issue

Metriche repository

Star
 (3702 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

As you can see in the switch expression, lastExperimentRunStatus === ExperimentRunStatus.RUNNING and lastExperimentRunStatus === ExperimentRunStatus.QUEUED are always true. IMO, we can shift these statements to true

Anyone who finds good first issues or participates in hacktoberfest can take this issue.

https://github.com/litmuschaos/litmus/blob/794676f6b75e08835334fd5d675e0b0d096d5f9d/chaoscenter/web/src/views/ExperimentDashboardV2/ExperimentDashboardV2Table.tsx#L207-L223

Guida contributor

Direzione di ricerca
Esamina l'espressione switch in ExperimentDashboardV2Table.tsx righe 207-223. Identifica le condizioni che sono sempre vere (ad esempio `lastExperimentRunStatus === ExperimentRunStatus.RUNNING` e `QUEUED`). Sostituisci tali condizioni con `true` e rimuovi i controlli ridondanti. Assicurati che la logica rimanga corretta.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85