pnpm run lint does not lint the e2e tests
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- eslint, typescript
Línea de trabajo
Empieza con el script de lint en package.json y e2e/cli-publish.test.ts:109; ejecuta el comando de ESLint indicado para confirmar los errores existentes. Comprueba cómo organizan los paquetes de libs/ sus carpetas e2e/ y, después, amplía la cobertura de lint y corrige las aserciones non-null indicadas. La tarea estará completa cuando pnpm run lint compruebe las pruebas end-to-end de la raíz y de las bibliotecas y se ejecute correctamente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Which package is this bug report for?
@nanoforge-dev/cli
Which command is this bug report for?
other
Issue description
pnpm run lint runs eslint on src only (package.json#L47: prettier --check . && eslint --format=pretty src), so e2e/ is never linted. It already holds lint errors:
pnpm exec eslint e2e/cli-publish.test.ts
109:12 error Forbidden non-null assertion @typescript-eslint/no-non-null-assertion
(cli-publish.test.ts#L109, reported twice on that line.) Found while reviewing the Windows test fixes (#274).
Expected: the lint script covers e2e/ (and the libs' e2e/ folders), and the existing errors are fixed.
Code sample
// e2e/cli-publish.test.ts:109 (main)
expect(put!.files!.sort()).toEqual(["index.ts", "nanoforge.manifest.json", "src/player.ts"]);
Versions
- @nanoforge-dev/cli
main(e95b881) - Node.js 26, Linux
Issue priority
Low (slightly annoying)
I have tested this issue on a development release
e95b881
- Lenguaje dominante
- TypeScript
- Estrellas
- 2
- Forks
- 2
- Merge medio
- 2 d 18 h
- PR fusionados (30 d)
- 17
Preparar el entorno
- Sin Dockerfile ni 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 NanoForge-dev/CLI
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
NanoForge-dev/CLI#294 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
NanoForge-dev/CLI#292 ·
Los mantenedores suelen responder en 1 día
-
command:dev enhancement good first issue refactor
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
NanoForge-dev/CLI#281 ·
Los mantenedores suelen responder en 1 día
-
bug command:build good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
NanoForge-dev/CLI#276 ·
Los mantenedores suelen responder en 1 día
-
documentation good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 74/100
NanoForge-dev/CLI#228 ·
Los mantenedores suelen responder en 1 día
Todos los issues de NanoForge-dev/CLI
Issues similares
-
Mend: dependency security vulnerability
Dificultad 1/5 1-3 horas Aptitud para principiantes 65/100
fdm-monster/fdm-monster#5563 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
be-BOP-io-SA/be-BOP#2849 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
Los mantenedores suelen responder en 1 día
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
growilabs/growi#12246 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
🐛 The Rulekeeper - An index run that gives up on the lock ends SucceededPosiblemente ocupada @github-actions la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
SSWConsulting/SSW.Rules#2809 · 1 comentario ·
Los mantenedores suelen responder en 1 día