calculateTree() runs forever if PR has 1000+ files
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- frontend
Línea de trabajo
Empieza en index.tsx, en calculateTree(), e inspecciona cómo se analiza files_tab_counter antes de comparar el número de archivos. Verifica la corrección con un pull request que contenga 1,000 archivos o más, y confirma que calculateTree() ya no se vuelve a activar indefinidamente cuando GitHub muestra un recuento separado por comas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
In index.tsx calculateTree() you parseInt the contents of files_tab_counter, but GitHub formats it with a comma (e.g. 1024 -> "1,024"), so parseInt returns the value before the first comma.
As a result, count is always greater than fileCount and we retrigger calculateTree over and over again:
if (fileCount !== count) {
setTimeout(this.calculateTree.bind(this), 100)
}
- Lenguaje dominante
- HTML
- Estrellas
- 697
- Forks
- 79
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 berzniz/github_pr_tree
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
berzniz/github_pr_tree#232 ·
-
Broken for Github EnterpriseAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
berzniz/github_pr_tree#229 · 1 comentario ·
-
firefox link is brokenAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 45/100
berzniz/github_pr_tree#228 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
berzniz/github_pr_tree#221 · 1 comentario · 3 reacciones ·
-
Fuzzy text search matchingAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
berzniz/github_pr_tree#217 ·
Todos los issues de berzniz/github_pr_tree
Issues similares
-
Dificultad 2/5 Menos de una hora Aptitud para principiantes 80/100
nuxt-content/nuxt-studio#599 ·
Los mantenedores suelen responder en 1 día
-
Plugin: sheet music can't be scrolled by hand before playbackPosiblemente ocupada @snakajima la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
receptron/jinglescript#51 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
open-webui/open-webui#33279 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 9 días