Refactoring core.ts
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
- #838 de @BassemHalim — cerrado sin fusionar
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Refactorización
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- developer-experience
Línea de trabajo
Comienza leyendo src/livecodes/core.ts y revisando sus 98 funciones, identificadas mediante el comando grep indicado en el issue. Define una división en módulos como ConfigManager, EditorManager o StorageManager y conserva las funciones existentes sin cambios de comportamiento; el trabajo estará completo cuando los módulos acordados se hayan extraído de core.ts.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi,
While working on another PR I noticed that core.ts was extremely long and it was a little hard to figure out how to implement some changes so I am planning on refactoring that code into multiple modules to make it more maintainable.
For example, core.ts has 98 functions
$ grep -c "= () =>" src/livecodes/core.ts
98
So I am thinking we could extract some modules from core.ts like a ConfigManager, EditorManager, StorageManger, etc. without changing the functions and then later we can refactor the long functions.
This might be too much for a single PR so maybe break the PRs into something like extracting ConfigManger from core.ts, extracting x etc.
What do you think?
- Lenguaje dominante
- TypeScript
- Estrellas
- 1.5k
- Forks
- 267
- Merge medio
- 7 h 47 min
- PR fusionados (30 d)
- 14
Preparar el entorno
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.
Más de live-codes/livecodes
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
live-codes/livecodes#1079 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 50/100
live-codes/livecodes#1060 · 5 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
live-codes/livecodes#1009 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Release a docker imageAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
live-codes/livecodes#1004 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
I18n: HungarianAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
live-codes/livecodes#980 · 6 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de live-codes/livecodes
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
pending close triaging
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
anomalyco/opencode#62582 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
feature request
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
callstack/react-native-paper#5189 ·
Los mantenedores suelen responder en 1 día
-
Add: ESP 2 ROAbiertoapproved check:passed streams:add
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día