Obsolete functions or functions that can be simplified.
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Lua
- Estrellas
- 8
- Forks
- 5
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
Merge Rooms : No longer necessary, since the hash from gmcp.room.info positively identifies a room, no need to merge rooms, since different rooms will have different hashes and thus different room ID's.
function set_room : At the end of this functions, an onMoveMap event is raised... yet you have no handler to handle the event. Is this an event handled by the mapper that I haven't found?
function check_room(...) : Since rooms are positively identified by a hash, why do you check against room name and exits? Big issue I have with this, is that some rooms have exits that are not obvious. Once I use said exit, the exit is created, and thus this function no longer identifies a perfectly valid room and won't show you being in that room.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza localizando las funciones Merge Rooms, set_room y check_room; después, busca handlers o referencias al evento onMoveMap. Verifica cómo los hashes de gmcp.room.info identifican las salas y cómo se representan las salidas; se considera terminado cuando las fusiones y comprobaciones obsoletas se hayan eliminado o simplificado sin impedir que se reconozcan las salas válidas.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- lua
- Área
- tooling
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 28/100