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 (528 Forks)batch import
good first issuepkg: corestatus: claimedtype: enhancement

Repository-Metriken

Stars
 (9.168 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

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

Research-Richtung
Überprüfe die verlinkte Unit.js Datei um Zeile 98, um zu verstehen, wie Effekte ausgeführt werden. Verfolge die asynchrone Methode und identifiziere, wo nach den Effekten eine Überprüfung der Lebenspunkte der Einheit hinzugefügt werden muss. Erwäge eine vorzeitige Rückkehr, wenn die Lebenspunkte null oder niedriger sind.
Tech Stack
javascript
Domain
backend
Issue Type
Fehler
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.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75