Remove commented-out code flagged by Sonar (java:S125)
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
先打开链接的 SonarCloud java:S125 列表，选择一个模块（例如 reader 或 calcite），然后在 issue 中评论以认领。阅读两个指定示例 SerializedArrayBoundsSecurityTest:126 和 PostscriptParserDTypeGuardsTest:194，了解两种情况：应删除的死代码，以及包含真实测试理由、需要改写为文字说明的代码块。创建分支前先阅读 CONTRIBUTING.md。完成标准是：该模块不再有 S125 代码块，每处删除或改写为文字说明都在 PR 中说明理由，并且没有留下被注释掉的代码。
由索引模型根据 Issue 内容生成。
描述
SonarCloud flags 21 blocks of commented-out code, all in tests: reader 8, calcite 6, integration 3, writer 2, csv 1, jdbc 1 (e.g. SerializedArrayBoundsSecurityTest:126, PostscriptParserDTypeGuardsTest:194). List: java:S125 open issues.
How: pick one module, say so in a comment here, and send one PR. Read each block first: delete it if it is dead, but if it documents a real case (why the data was chosen, what bug a test catches) rewrite it as a prose comment explaining the why. Do not leave code in comments. See CONTRIBUTING.md.
- 主要语言
- Java
- 星标
- 3
- 派生
- 2
- 平均合并
- 9 小时 19 分钟
- 30 天内合并 PR
- 154
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
dfa1/vortex-java 的其他 Issue
-
good first issue
难度 2/5 1-3 小时 新手友好度 70/100
dfa1/vortex-java#738 ·
维护者通常 1 天内回复
-
good first issue
难度 2/5 1-3 小时 新手友好度 78/100
dfa1/vortex-java#736 ·
维护者通常 1 天内回复
-
help wanted project
难度 5/5 一周以上 新手友好度 25/100
dfa1/vortex-java#729 ·
维护者通常 1 天内回复
-
enhancement good first issue
难度 3/5 1-2 天 新手友好度 72/100
dfa1/vortex-java#728 ·
维护者通常 1 天内回复
-
bug
难度 4/5 3-5 天 新手友好度 55/100
dfa1/vortex-java#723 ·
维护者通常 1 天内回复
相似的 Issue
-
难度 1/5 1 小时以内 新手友好度 88/100
-
难度 2/5 1 小时以内 新手友好度 84/100
cpp-best-practices/project_validator#251 ·
维护者通常 1 天内回复
-
backend enhancement
难度 2/5 半天 新手友好度 70/100
damodharg6/payledger#16 ·
-
[Bug]: An initiative list with no current entry logs an error on every change可能已有人在做 @dwizard0 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 75/100
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 85/100
维护者通常 1 天内回复