Track TypeScript errors in CI
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
- #8787 de @talissoncosta — cerrado sin fusionar
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- github-actions, typescript
- Área
- ci-cd
Línea de trabajo
Start by reading the existing CI workflows to see how lint and unit tests run, and reuse the sticky PR comment already built for Visual Regression. Run tsc --noEmit locally to reproduce the per-file error counts, starting with common/stores/feature-list-store.ts (147 errors). Done when a PR shows the 890 to N comparison against main's merge base and the workflow never blocks a merge.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
CI runs lint and unit tests but not tsc. Main has about 890 type errors across 351 files, and nothing tells us when a PR adds more.
Add a report on every PR that compares its type errors with main's, without blocking the merge.
- On push to main: run
tsc --noEmit, upload per-file error counts as an artifact - On PRs: run
tsconce, compare with main's counts for the merge base (fall back to the latest main) - Sticky PR comment, like the Visual Regression one:
- 🟢 "Type errors: 890 → 884 (−6)" when the PR fixes some
- 🔴 "Type errors: 890 → 892 (+2)", listing the files that gained errors
- nothing to report when unchanged
- Not a required check, so it never blocks a merge
Later, once it has proved quiet: consider making it required.
The biggest single file is common/stores/feature-list-store.ts (147 errors), which shrinks with the Flux migration.
- Lenguaje dominante
- Python
- Estrellas
- 7.1k
- Forks
- 603
- Merge medio
- 3 d 8 h
- PR fusionados (30 d)
- 102
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 Flagsmith/flagsmith
-
Expose whether an organisation is blocked for API usagePosiblemente ocupada @PiUnknown la tomó hace 26 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Flagsmith/flagsmith#8773 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Environment colors mismatch in audit log viewPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Flagsmith/flagsmith#8681 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
influx_query_manager doesn't catch ApiException, so Influx error responses 500Posiblemente ocupada @SAY-5 la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Flagsmith/flagsmith#8510 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Sales dashboard: download_org_data 500s with "Object of type generator is not JSON serializable"Posiblemente ocupada @Iflaqbhat la tomó hace 46 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
Todos los issues de Flagsmith/flagsmith
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
-
area:other help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
WingedGuardian/GENesis-AGI#4734 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
peach-mitao/peach#693 ·
Los mantenedores suelen responder en 1 día
-
Failed result serialization destroys a previously saved JSON resultPosiblemente ocupada @chessing234 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
huggingface/evaluate#867 ·
-
FTP reads inside a transaction use a nonexistent staging pathPosiblemente ocupada @Harsh23Kashyap la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
fsspec/filesystem_spec#2463 ·
Los mantenedores suelen responder en 1 día