Remove commented-out code flagged by Sonar (java:S125)
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 70/100
Research direction
Start by opening the linked SonarCloud java:S125 list and picking a single module (e.g. reader or calcite), then comment on the issue to claim it. Read the two named examples, SerializedArrayBoundsSecurityTest:126 and PostscriptParserDTypeGuardsTest:194, to see the two cases: dead code to delete versus a block encoding a real test rationale to rewrite as prose. Read CONTRIBUTING.md before branching. Done means that module has no remaining S125 blocks, each deletion or prose rewrite is justified in the PR, and no commented-out code is left behind.
Written by the indexing model from the issue text.
Description
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.
- Dominant language
- Java
- Stars
- 3
- Forks
- 2
- Avg merge
- 9h 19m
- Merged PRs (30d)
- 154
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from dfa1/vortex-java
-
good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
dfa1/vortex-java#738 ·
Maintainers usually reply within 1 day
-
good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
dfa1/vortex-java#736 ·
Maintainers usually reply within 1 day
-
help wanted project
Difficulty 5/5 Over a week Newbie friendliness 25/100
dfa1/vortex-java#729 ·
Maintainers usually reply within 1 day
-
enhancement good first issue
Difficulty 3/5 1-2 days Newbie friendliness 72/100
dfa1/vortex-java#728 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 4/5 3-5 days Newbie friendliness 55/100
dfa1/vortex-java#723 ·
Maintainers usually reply within 1 day
All issues in dfa1/vortex-java
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
-
Difficulty 2/5 Under an hour Newbie friendliness 84/100
cpp-best-practices/project_validator#251 ·
Maintainers usually reply within 1 day
-
backend enhancement
Difficulty 2/5 Half a day Newbie friendliness 70/100
damodharg6/payledger#16 ·
-
[Bug]: An initiative list with no current entry logs an error on every changePossibly taken @dwizard0 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 85/100
Maintainers usually reply within 1 day