.d.ts files should be forbidden from importing .ts files
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
- 55/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- build-system
Línea de trabajo
En types/adobe__pdfservices-node-sdk, ejecuta npm test adobe__pdfservices-node-sdk después de cambiar el nombre de client-config.d.ts a client-config.ts, tal como se muestra. Sigue la validación alrededor de Index.d.ts y del client-config.ts renombrado, y haz que la prueba informe del error de importación esperado en lugar del error de compilación actual.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
.d.ts files should not be able to import .ts files; .ts files are supposed to be only for testing. But it's too easy to make a mistake and use .ts for a file intended for shipping types. See for example https://discord.com/channels/508357248330760243/757992230077333655/1086956594736148511
To repro:
cd $dt/types/adobe__pdfservices-node-sdk
git mv client-config.d.ts client-config.ts
npm test adobe__pdfservices-node-sdk
Expected: Error "Index.d.ts can't import from client-config.ts because it's a .ts file"
Actual: Compile error because client-config.ts uses syntax that is illegal in .ts (function signature without declare or following implementation signature).
- Lenguaje dominante
- TypeScript
- Estrellas
- 422
- Forks
- 237
- Merge medio
- 1 d 4 h
- PR fusionados (30 d)
- 8
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 microsoft/DefinitelyTyped-tools
-
Dificultad 3/5 1-2 días Aptitud para principiantes 66/100
microsoft/DefinitelyTyped-tools#1324 · 8 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 32/100
microsoft/DefinitelyTyped-tools#1230 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
microsoft/DefinitelyTyped-tools#1229 ·
Los mantenedores suelen responder en 1 día
-
mergebot staleness comments do not respect tooManyOwnersPosiblemente ocupada @copilot-swe-agent la tomó hace 327 días. Abierto
microsoft/DefinitelyTyped-tools#1218 · 1 reacción · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
microsoft/DefinitelyTyped-tools#1211 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de microsoft/DefinitelyTyped-tools
Issues similares
-
area:deploy task
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
wilanis/wilanis-js#1017 ·
Los mantenedores suelen responder en 1 día
-
documentation v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
modelcontextprotocol/servers#5133 ·
Los mantenedores suelen responder en 1 día
-
budget bug feature frontend good-issue medium needs-review OSCI'26 payments
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
SatyamPandey-07/WorkSphere#5990 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
lugassawan/swe-workbench#799 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue javascript
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
yukikotani231/sqlsift#326 ·
Los mantenedores suelen responder en 1 día