Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-36720

Investigate/cleanup medium-severity FindBugs issues in the Jenkins core

    XMLWordPrintable

Details

    Description

      Currently FindBugs settings ignore all medium-priority issues. It's bad, because this level contains NPE checks and other serious issues.

      TODOs:

      • Remove filtering of medium-priority issues by setting <findbugs.threshold>Medium</findbugs.threshold>
      • Investigate error codes, add low-important once to the ignore list
      • Fix important issues in small pull requests

      This task can be handled by multiple contributors!

      Attachments

        Activity

          Current status:

          [INFO] Total bugs: 400

          stefan_spieker Stefan Spieker added a comment - Current status: [INFO] Total bugs: 400
          oleg_nenashev Oleg Nenashev added a comment -

          Ugh. But it is still a  very good progress, thanks!

          oleg_nenashev Oleg Nenashev added a comment - Ugh. But it is still a  very good progress, thanks!

          We are on a good way, but still some steps to go:

          [java] Warnings generated: 362

           

          stefan_spieker Stefan Spieker added a comment - We are on a good way, but still some steps to go: [java] Warnings generated: 362  
          oleg_nenashev Oleg Nenashev added a comment -

          Thanks for your hard work stefan_spieker! I really look forward to introduce stricter requirements at some point

          oleg_nenashev Oleg Nenashev added a comment - Thanks for your hard work stefan_spieker ! I really look forward to introduce stricter requirements at some point

          Probably due to findsecbugs we are at [java] Warnings generated: 591 right now

          raihaan Raihaan Shouhell added a comment - Probably due to findsecbugs we are at [java] Warnings generated: 591 right now

          People

            stefan_spieker Stefan Spieker
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated: