microsoft / microsoft/TypeScript
Double slash comment `//` will be wrongly removed in the trailing position of the object attribute
未关闭
还没有人认领这个 Issue。
Domain: Comment Emit
Help Wanted
Possible Improvement
- 主要语言
- Go
- 星标
- 111k
- 派生
- 14.4k
- 平均合并
- 1 天 19 小时
- 30 天内合并 PR
- 117
描述
🔎 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
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
首先跟踪 issue 中提到的 removeComments 行为,并在链接的 TypeScript Playground 中重现该案例。比较对象末尾属性在有和没有后续逗号时的注释处理方式。完成的标准是:保留所示的 foo 注释,同时现有的 bar 注释保持不变,并为该示例添加一个回归测试。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- typescript
- 领域
- compilers
- Issue 类型
- 缺陷
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 48/100