jenkinsci/jenkinsfile-runner

Cleanup SpotBugs warnings in the codebase

Open

#374 aperta il 30 set 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Java (291 fork)batch import
good first issueinternal

Metriche repository

Star
 (1090 star)
Metriche merge PR
 (Merge medio 47g 2h) (3 PR mergiate in 30 g)

Descrizione

There is a number of SporBugs static analysis warnings in the old parts of the Jenkinsfile Runner codebase. It is possible to see the warnings in the build log. Although there is no severe issues, it would be great to cleanup these warnings and then enforce SpotBugs checks during the build.

List of the current SpotBugs violations: https://ci.jenkins.io/job/jenkinsfile-runner/job/jenkinsfile-runner/job/master/lastSuccessfulBuild/analysis/

Acceptance criteria:

  • All SpotBugs violations need to be either fixed or suppressed
  • FindBugs failOnError is reenabled in the pom.xml

NOTE: Contributors can submit pull requests for particular issues and issue types, no need to fix everything in a single pull request.

Guida contributor

Direzione di ricerca
Esamina l'elenco delle violazioni di SpotBugs, scegli un avviso specifico, comprendi il problema e correggilo o sopprimilo. Riattiva failOnError in pom.xml.
Tech stack
java
Dominio
build system
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaMavenSpotBugs
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75