Remove commented-out code flagged by Sonar (java:S125)
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 70/100
Direção de pesquisa
Comece abrindo a lista do SonarCloud java:S125 vinculada e escolha um único módulo (por exemplo, reader ou calcite). Depois, comente na issue para reivindicá-lo. Leia os dois exemplos citados, SerializedArrayBoundsSecurityTest:126 e PostscriptParserDTypeGuardsTest:194, para entender os dois casos: código morto a ser excluído, em contraste com um bloco que codifica uma justificativa real de teste e deve ser reescrito em prosa. Leia CONTRIBUTING.md antes de criar a branch. Está concluído quando esse módulo não tem mais blocos S125, cada exclusão ou reescrita em prosa está justificada no PR e nenhum código comentado foi deixado para trás.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Java
- Estrelas
- 3
- Forks
- 2
- Merge médio
- 9h 19min
- PRs com merge (30d)
- 154
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de dfa1/vortex-java
-
good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
dfa1/vortex-java#738 ·
Mantenedores costumam responder em até 1 dia
-
good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
dfa1/vortex-java#736 ·
Mantenedores costumam responder em até 1 dia
-
help wanted project
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
dfa1/vortex-java#729 ·
Mantenedores costumam responder em até 1 dia
-
enhancement good first issue
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
dfa1/vortex-java#728 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 55/100
dfa1/vortex-java#723 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de dfa1/vortex-java
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 80/100
finos/fluxnova-bpm-platform#387 ·
Mantenedores costumam responder em até 4 dias
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
-
Dificuldade 2/5 Menos de uma hora Facilidade para iniciantes 84/100
cpp-best-practices/project_validator#251 ·
Mantenedores costumam responder em até 1 dia
-
backend enhancement
Dificuldade 2/5 Meio dia Facilidade para iniciantes 70/100
damodharg6/payledger#16 ·
-
[Bug]: An initiative list with no current entry logs an error on every changeTalvez já em andamento @dwizard0 assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia