microsoft / microsoft/TypeScript
Double slash comment `//` will be wrongly removed in the trailing position of the object attribute
Ouverte
Personne n'a encore pris cette issue.
Domain: Comment Emit
Help Wanted
Possible Improvement
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.4k
- Merge moyen
- 1 j 19 h
- PR mergées (30 j)
- 117
Description
🔎 Search Terms
- comment removal
- removeComments
🕗 Version & Regression Information
- This is the behavior in every version I tried.
⏯ Playground Link
💻 Code
export const foo = {
foo: 'foo', // this comment will be removed, however this should be kept
bar: 'bar' // this comment will be kept, works well
}
🙁 Actual behavior
export const foo = {
foo: 'foo',
bar: 'bar' // this comment will be kept, works well
}
🙂 Expected behavior
export const foo = {
foo: 'foo', // this comment will be removed, however this should be kept
bar: 'bar' // this comment will be kept, works well
}
Additional information about the issue
No response
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
Commencez par retracer le comportement de removeComments mentionné dans l’issue et reproduisez le cas dans le TypeScript Playground lié. Comparez le traitement des commentaires pour l’attribut final de l’objet avec et sans virgule suivante. Le travail est terminé lorsque le commentaire foo montré est conservé tandis que le commentaire bar existant reste inchangé, avec un test de régression couvrant l’exemple.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- compilers
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 48/100