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

Ouverte Adaptée aux débutants
#737 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
70/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
Active
Stack technique
java
Domaine
testing

Piste de recherche

Commencez par ouvrir la liste SonarCloud java:S125 liée et choisissez un seul module (par exemple reader ou calcite), puis commentez le ticket pour le réserver. Lisez les deux exemples cités, SerializedArrayBoundsSecurityTest:126 et PostscriptParserDTypeGuardsTest:194, pour comprendre les deux cas : du code mort à supprimer, par opposition à un bloc qui encode une justification réelle de test à réécrire en prose. Lisez CONTRIBUTING.md avant de créer votre branche. C'est terminé lorsque ce module ne contient plus aucun bloc S125, que chaque suppression ou réécriture en prose est justifiée dans la PR et qu'aucun code commenté ne subsiste.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
Java
Étoiles
3
Forks
2
Merge moyen
9 h 19 min
PR mergées (30 j)
154

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de dfa1/vortex-java

Toutes les issues de dfa1/vortex-java

Issues similaires

Plus d'issues Java

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.