jenkinsci/jenkinsfile-runner

Cleanup SpotBugs warnings in the codebase

Open

#374 ouverte le 30 sept. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Java (291 forks)batch import
good first issueinternal

Métriques du dépôt

Stars
 (1 090 stars)
Métriques de merge PR
 (Merge moyen 47j 2h) (3 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez la liste des violations SpotBugs, choisissez un avertissement specifique, comprenez le probleme et corrigez ou supprimez le. Reactiver failOnError dans pom.xml.
Stack technique
java
Domaine
build system
Type d'issue
Maintenance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaMavenSpotBugs
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75