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

flex-pmd output not picked up when running as build plugin

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Major
    • Resolution: Fixed
    • pmd-plugin
    • None

    Description

      I am running flex-pmd as part of the build, not of the reporting:

      <plugin>
        <groupId>com.adobe.ac</groupId>
        <artifactId>flex-pmd-maven-plugin</artifactId>
        <executions>
          <execution>
            <phase>package</phase>
            <goals>
              <goal>report</goal>
            </goals>
          </execution>
        </executions>
      </plugin>
      

      Unfortunately the Jenkins PMD plugin does not trigger then because it seems to trigger only on the "pmd" and "site" goals, but not on "report" or "check" which are the two supported by the flex-pmd plugin. I would prefer not having to run an extra "site" step for the output to be picked up. Would it be possible to accept the "report" and "check" goals?

          @Override
          protected boolean acceptGoal(final String goal) {
              return "pmd".equals(goal) || "site".equals(goal) || "report".equals(goal) || "check".equals(goal);
          }
      

      I assume that fixes it, but I didn't test it. However, I didn't find any place where the plugin actually looks out for the name of Mojo.

      Attachments

        Activity

          rec Richard Eckart de Castilho created issue -
          scm_issue_link SCM/JIRA link daemon made changes -
          Field Original Value New Value
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 140523 ] JNJira + In-Review [ 189106 ]

          People

            drulli Ulli Hafner
            rec Richard Eckart de Castilho
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: