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

Confusing label for the list of regex that include and exclude files

    Details

    • Type: Patch
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: findbugs-plugin
    • Labels:
      None
    • Similar Issues:

      Description

      The labels "Bugs to include" and "Bugs to ignore" seem to refer to bugs that can be included or ignored.

      However, what they really specify is a list of regex that include or exclude files from being processed by findbugs, not bugs.

      Here's my proposed patch:

      diff --git a/plugin/src/main/resources/findbugs/filter.jelly b/plugin/src/main/resources/findbugs/filter.jelly
      — a/plugin/src/main/resources/findbugs/filter.jelly
      +++ b/plugin/src/main/resources/findbugs/filter.jelly
      @@ -6,10 +6,10 @@
      <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define"
      xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form">

      • <f:entry title="$ {%Bugs to include}

        " field="includePattern" description="$

        {%description.includePattern}">
        + <f:entry title="${%Files to include}" field="includePattern" description="${%description.includePattern}

        ">
        <f:textbox />
        </f:entry>

      • <f:entry title="$ {%Bugs to ignore}

        " field="excludePattern" description="$

        {%description.excludePattern}">
        + <f:entry title="${%Files to exclude}" field="excludePattern" description="${%description.excludePattern}

        ">
        <f:textbox />
        </f:entry>

        Attachments

          Activity

            People

            • Assignee:
              drulli Ulli Hafner
              Reporter:
              elifarley Elifarley
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: