microsoft / microsoft/TypeScript
Add extract to parameter refactoring for JS/TS in vscode
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.3k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 117
Descripción
Search Terms
Suggestion
Add a refactoring to extract to parameter for JS/TS
e.g.
for
const fangler = () => {
return {
someProperty: [
"a", "b"
]
}
}
the extract to parameter refactoring would change to
const fangler = (secondItem) => {
return {
someProperty: [
"a", secondItem
]
}
}
and populate each existing call site with the extracted value (in this example "b")
Use Cases
It is a common refactoring when you discover that a previously fixed value should instead be provided by a caller
Examples
e.g. in the case where I went looking for it code in a Lambda function that created a fixed header had to start providing a varying value depending on context and the easiest refactoring was to extract the currently fixed value as a parameter so that the callers that would vary it could do without breaking other callers
see https://twitter.com/mattbierner/status/1234920525543002113
Checklist
My suggestion meets these guidelines:
- This wouldn't be a breaking change in existing TypeScript/JavaScript code
- This wouldn't change the runtime behavior of existing JavaScript code
- This could be implemented without emitting different JS based on the types of the expressions
- This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
- This feature would agree with the rest of TypeScript's Design Goals.
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 revisando la compatibilidad existente con la refactorización de JavaScript/TypeScript en la integración de VS Code y el comportamiento de ejemplo descrito en el issue. Define cómo una expresión extraída se convierte en un parámetro y cómo cada sitio de llamada existente recibe el valor original; se considera terminado cuando la refactorización funciona tanto para JS como para TS sin cambiar el comportamiento del tiempo de ejecución no relacionado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, typescript
- Área
- developer-experience, devtools
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 32/100