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

Mark warnings with an inspection status

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      I think it would be perfect that if users could handle warnings. Warnings could be defaultly uninspected status. After users inspectation, user could mark it bug, intentional, or false positive.

      If some warnings are not bug, the next build will not identify them as errors.

        Attachments

          Issue Links

          duplicates

          Improvement - An improvement or enhancement to an existing feature or task. JENKINS-732 Classification of findbug issues

          • Trivial - Cosmetic problem like misspelt words or misaligned text.
          • Resolved
          is related to

          Improvement - An improvement or enhancement to an existing feature or task. JENKINS-732 Classification of findbug issues

          • Trivial - Cosmetic problem like misspelt words or misaligned text.
          • Resolved

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            drulli Ulli Hafner added a comment - - edited

            This is related to JENKINS-732. The FindBugs team currently develops something similar: http://findbugs.cs.umd.edu/cloud/.

            Show
            drulli Ulli Hafner added a comment - - edited This is related to JENKINS-732 . The FindBugs team currently develops something similar: http://findbugs.cs.umd.edu/cloud/ .
            Hide
            Permalink
            jasperlj jasperlj added a comment -

            Thanks for your information.

            I had a look at JENKINS-732. I suggest not to suppress warning in source code, as we have to modify all parsers. I advise that, false positive warnings could be marked false in UI, and later warnings could be compared to it. So false ones could go away in later build.

            Show
            jasperlj jasperlj added a comment - Thanks for your information. I had a look at JENKINS-732 . I suggest not to suppress warning in source code, as we have to modify all parsers. I advise that, false positive warnings could be marked false in UI, and later warnings could be compared to it. So false ones could go away in later build.
            Hide
            Permalink
            jasperlj jasperlj added a comment -

            Could the feature come to analysis-core or plug-ins based on analysis-core?
            And could you predict the time? Thanks!

            Show
            jasperlj jasperlj added a comment - Could the feature come to analysis-core or plug-ins based on analysis-core? And could you predict the time? Thanks!
            Hide
            Permalink
            drulli Ulli Hafner added a comment -

            This makes definitely sense in the core, as all plug-ins could benefit.

            I don't think that I have time to implement such a feature. Are you interested in providing a patch?

            Show
            drulli Ulli Hafner added a comment - This makes definitely sense in the core, as all plug-ins could benefit. I don't think that I have time to implement such a feature. Are you interested in providing a patch?
            Hide
            Permalink
            jasperlj jasperlj added a comment -

            Sorry, I don't have much experience on developing such kind of complicated plug-in.
            I even don't know where to start. But as you said, this feature could benefit all analysis tools.
            Could any body help to implement it? Or do you have any idea in mind how to implement it?

            Thanks!

            Show
            jasperlj jasperlj added a comment - Sorry, I don't have much experience on developing such kind of complicated plug-in. I even don't know where to start. But as you said, this feature could benefit all analysis tools. Could any body help to implement it? Or do you have any idea in mind how to implement it? Thanks!

              People

              Assignee:
              drulli Ulli Hafner
              Reporter:
              jasperlj jasperlj
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Dates

                Created:
                Updated: