olistic/warriorjs

Check if unit is alive after effects

Open

#63 geöffnet am 8. Mai 2018

Auf GitHub ansehen
 (8 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (9.168 Stars) (528 Forks)batch import
good first issuepkg: corestatus: claimedtype: enhancement

Beschreibung

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.

Contributor Guide

Tech Stack
javascript
Domain
backend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Familiarity with WarriorJS game logicUnderstanding of Unit.js effects execution
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85
Research-Richtung
Examine the Unit.js file in packages/warriorjs core/src, specifically the method that executes effects around line 98. Understand the current flow and add a check after calling the effects to verify the unit's health. If health is 0 or below, cause the unit to die or return early. This is a small localized change.