microsoft / microsoft/TypeScript
importing merged declorations in js files
Abierto
Nadie ha tomado este issue todavía.
Bug
Domain: JSDoc
Help Wanted
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.3k
- Merge medio
- 2 d 4 h
- PR fusionados (30 d)
- 132
Descripción
🔎 Search Terms
is a type and cannot be imported in JavaScript files;
JSDoc;
Import;
Declaration Merging.
🕗 Version & Regression Information
- This is the behavior in every version I tried (v5.8.3 - Nightly)
💻 Code
// @filename: foo.impl.js
/**
* @import { Foo } from "./foo.js"
*/
/**
* @returns { Foo }
*/
export function alloc() {
return {
prop1: 42,
prop2: "lorem ipsum"
}
}
// @filename: foo.js
export * as Foo from "./foo.impl.js"
/**
* @typedef Foo
* @prop { number } prop1
* @prop { string } prop2
*/
// @filename: index.js
import { Foo } from "./foo.js"
// ^^^ 'Foo' is a type and cannot be imported in JavaScript files. Use 'import("./foo.js").Foo' in a JSDoc type annotation.ts(18042)
const foo = Foo.alloc();
🙁 Actual behavior
tsc/lsp emmits error
🙂 Expected behavior
no error
Additional information about the issue
same example works fine using ts files
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.
Línea de trabajo
Reproduce el ejemplo usando foo.impl.js, foo.js e index.js con tsc o el language service; luego rastrea el diagnóstico de la importación de Foo y compáralo con el caso funcional del archivo TypeScript. Se considera terminado cuando el ejemplo de JavaScript no informa de ningún error de importación y Foo.alloc() sigue siendo utilizable.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, typescript
- Área
- compilers
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100