TauCetiStation/TauCetiClassic

[Task] Рефакторинг телепортов

Open

#9.831 geöffnet am 15. Aug. 2022

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)DM (433 Forks)batch import
Good First IssueTask

Repository-Metriken

Stars
 (153 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 19T 19h) (63 gemergte PRs in 30 T)

Beschreibung

В билде есть утилитарный do_teleport и вариации /datum/teleport, но не все телепорты в игре используют эти инструменты. В результате, у нас куча независимых методов, каждый из которых во многом делает то же, что и другие, повторяя код, и иногда лучше или хуже.

Нужно убрать копипасту и свести всё к do_teleport, по необходимости дописать функционал. Стоит еще смотреть на код с /tg/, как пример.

Многие телепорты в билде сейчас можно найти по COMSIG_ATOM_INTERCEPT_TELEPORT.

Contributor Guide

Research-Richtung
Die bestehenden Funktionen do teleport und /datum/teleport sind in der Codebasis zu finden. Das Signal COMSIG ATOM INTERCEPT TELEPORT wird verwendet, um Teleportationen abzufangen; eine Suche danach wird den relevanten Code aufzeigen. Das Refactoring sollte alle teleportbezogenen Funktionen identifizieren und sicherstellen, dass sie do teleport verwenden, möglicherweise durch Erweiterung. Es wird empfohlen, die /tg/ Codebasis auf ähnliche Implementierungen zu überprüfen.
Tech Stack
Keine
Domain
backend
Issue Type
Refactoring
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.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70