microsoft / microsoft/TypeScript
Preserve type aliases on implement interface
Abierto
Nadie ha tomado este issue todavía.
Experience Enhancement
Suggestion
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.3k
- Merge medio
- 2 d 4 h
- PR fusionados (30 d)
- 132
Descripción
From https://github.com/microsoft/vscode/issues/90061
TypeScript Version: 3.8.1-rc
Search Terms:
- quick fix
- implement interface
Repo
export type UrlString = string;
export type UUID = string;
export interface I_IdToUrlMapper {
mapIdToUrl(id: UUID): UrlString
mapUrlToId(url: UrlString): UUID
}
export class IdToUrlMapper implements I_IdToUrlMapper {
}
Run implement interface on IdToUrlMapper
Expected behavior:
Type aliases are preserved
export class IdToUrlMapper implements I_IdToUrlMapper {
mapIdToUrl(id: UUID): UrlString {
throw new Error("Method not implemented.");
}
mapUrlToId(url: UrlString): UUID {
throw new Error("Method not implemented.");
}
}
Actual behavior:
Both methods use string:
export class IdToUrlMapper implements I_IdToUrlMapper {
mapIdToUrl(id: string): string {
throw new Error("Method not implemented.");
}
mapUrlToId(url: string): string {
throw new Error("Method not implemented.");
}
}
Related Issues
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
Comienza con la corrección rápida “implement interface” del TypeScript language-service usando el ejemplo reproducido de IdToUrlMapper. Compara las firmas de método generadas con la salida esperada y verifica que los alias de tipo como UUID y UrlString se conserven en lugar de reducirse a string.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- tooling
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 55/100