A warning located in an external template fails the module

Abierto
#32 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
68/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
angular, typescript, vite

Línea de trabajo

Read the compilerPlugin warning path and inspect dist/index.js, where the issue proposes changing how diagnostics are passed to Vite. Reproduce the case with a warning located in foo.component.html and check how Vite maps its position against the .ts module. Done means the warning is reported without causing the module to fail.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug

Part of #27.

Problem

compilerPlugin passes a diagnostic located in foo.component.html to this.warn(log) with its pos. Vite reads pos against the .ts module's code and throws when the offset is past its end, so a warning fails the module.

Proposed change

When log.id !== file, warn with a message only, for example `${log.id}: ${log.message}` (dist/index.js). Better still, report the template location in a form Vite doesn't map onto the module.

Setup: ngp 0.3.0, @angular/compiler/compiler-cli 22.3.0-next.4, Vite 8, Vitest 5.1.0 (pool: 'forks'), a Nx monorepo with 1,035 Angular projects, one Vitest config each, specs written for TestBed under JIT (Analog).

Lenguaje dominante
TypeScript
Estrellas
7
Forks
0
Merge medio
12 h 24 min
PR fusionados (30 d)
13

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

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de push-based/vite-plugin-ngp

Todos los issues de push-based/vite-plugin-ngp

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.