microsoft / microsoft/TypeScript
Rename field to a name with dash does not transform the field with quotes.
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.4k
- Merge moyen
- 1 j 19 h
- PR mergées (30 j)
- 117
Description
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
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Reproduisez le renommage dans VS Code à l’aide de l’exemple TypeScript, en remplaçant test par te-st. Suivez le comportement du refactoring de renommage pour la propriété de l’interface, le littéral d’objet et l’accès au membre ; le travail est terminé lorsque toutes les références concernées utilisent des propriétés entre guillemets valides et un accès par crochets, comme indiqué dans la sortie attendue, et qu’une couverture de régression a été ajoutée dans la suite de tests concernée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- devtools
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100