olistic/warriorjs

Check if unit is alive after effects

Open

#63 aperta il 8 mag 2018

Vedi su GitHub
 (8 commenti) (0 reazioni) (0 assegnatari)JavaScript (528 fork)batch import
good first issuepkg: corestatus: claimedtype: enhancement

Metriche repository

Star
 (9168 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

A negative effect could potentially kill a unit. We need to check if the unit is still alive after executing the effects and return early if the unit died. Example: a "poisoned" effect (it doesn't exist, at least yet) that damages the unit turn by turn.

Guida contributor

Direzione di ricerca
Ispeziona il file Unit.js collegato intorno alla riga 98 per capire come vengono eseguiti gli effetti. Traccia il metodo asincrono e identifica dove aggiungere un controllo per la salute dell'unità dopo gli effetti. Considera di restituire anticipatamente se la salute è zero o inferiore.
Tech stack
javascript
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75