microsoft / microsoft/TypeScript
Rename field to a name with dash does not transform the field with quotes.
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
TypeScript Version: 3.5.0-dev.20190412
Search Terms: hyphen refactor, dash rename
Code
interface TestObject {
test:string
}
const instance: TestObject = { test: "hello" };
console.log(instance.test);
Rename "test" to a variable with a hyphen, e.g "te-st"
Expected behavior:
Code is refactored to use quotes:
interface TestObject {
"te-st":string
}
const instance: TestObject = { "te-st": "hello" };
console.log(instance["te-st"]);
Actual behavior:
Refactor replaces the variable as given, producing invalid code
interface TestObject {
te-st:string
}
const instance: TestObject = { te-st: "hello" };
console.log(instance.te-st);
Playground Link: Replacing all occurrences in the playground also includes all Test matches, so I tested this in vscode
Related Issues: none I could find
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 renombrado en VS Code usando el ejemplo de TypeScript, cambiando test por te-st. Rastrea el comportamiento de la refactorización de renombrado para la propiedad de la interfaz, el literal de objeto y el acceso al miembro; se considera terminado cuando todas las referencias afectadas usan propiedades entre comillas válidas y acceso mediante corchetes, tal como se muestra en la salida esperada, y se ha añadido cobertura de regresión en la suite de pruebas correspondiente.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- devtools
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100