microsoft / microsoft/TypeScript
In scripthost.d.ts add `WScript` interface for extendibility
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.3k
- Merge medio
- 2 d 4 h
- PR fusionados (30 d)
- 132
Descripción
lib Update Request
I want to write a type lib to extend WScript, allow CreateObject to recognize some specified ProgId, and make the returned COMObject typed:
var app = WScript.CreateObject("Microsoft.Excel");
// ^^
// make this typed
But in scripthost.d.ts, The type of WScript is inlined with it's declearation:
declare var WScript: { /* inlined type definition */ };
To make it possible to extend, we can add a interface using the same name WScript and set this as the type of the var WScript:
interface WScript { /* type definition of WScript */ }
declare var WScript: WScript;
In thiry part lib, they could extend WScript var by extends the interface WScript. for example, In my lib, I could write:
interface WScript {
CreateObject("Microsoft.Excel"): Excel.App;
}
declare namespace Excel {
interface App {
//...
}
}
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 en scripthost.d.ts localizando la declaración inline de WScript y la firma de CreateObject. Refactoriza la declaración para que WScript sea una interfaz utilizada por la variable global y confirma que una ampliación de interfaz de terceros puede especializar CreateObject para un ProgId especificado y devolver un objeto COM tipado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- tooling
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 52/100