Remove commented-out code flagged by Sonar (java:S125)

Abierto Apto para principiantes
#737 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
70/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
java
Área
testing

Línea de trabajo

Empieza abriendo la lista de SonarCloud java:S125 enlazada y elige un único módulo (por ejemplo, reader o calcite). Luego comenta en el issue para reclamarlo. Lee los dos ejemplos indicados, SerializedArrayBoundsSecurityTest:126 y PostscriptParserDTypeGuardsTest:194, para entender los dos casos: código muerto que hay que eliminar frente a un bloque que codifica una justificación real de un test y que hay que reescribir como prosa. Lee CONTRIBUTING.md antes de crear la rama. Se considera terminado cuando ese módulo no tiene bloques S125 pendientes, cada eliminación o reescritura en prosa está justificada en el PR y no queda código comentado.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

good first 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.

Lenguaje dominante
Java
Estrellas
3
Forks
2
Merge medio
9 h 19 min
PR fusionados (30 d)
154

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de dfa1/vortex-java

Todos los issues de dfa1/vortex-java

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.