Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Incomplete
    • 2.9.1
    • None
    • None
    • None
    • Apache Maven 3.0.4 (r1232337; 2012-01-17 09:44:56+0100)
      Java version: 1.6.0_33, vendor: Sun Microsystems Inc.
      OS name: "linux", version: "3.2.0-36-generic", arch: "amd64", family: "unix"

    Description

      When I have module configured as war (packaging war). Checkstyle plugin should check for all file in project i.e.

      • /src/main/java
      • /src/main/resources
      • /src/main/webapp

      when includeTestSourceDirectory is enabled, chekckstyle plugin should add more paths like:

      • /src/test/java
      • /src/test/resources
      • /src/test/webapp

      Attachments

        Issue Links

          Activity

            michael-o Michael Osipov added a comment -

            Solution was offered, reporter did not respond in a decent amount of time.

            michael-o Michael Osipov added a comment - Solution was offered, reporter did not respond in a decent amount of time.
            ppalaga Peter Palaga added a comment -

            I was about to vote this up but then I found a config that works for me and probably will work also for the reporter.

            My use case is that I want to check tabs and trailing spaces in pom.xml files. Simply setting <sourceDirectory>${basedir}</sourceDirectory> includes pom.xml but also pulls all submodule directories in, which I do not want. To get rid of the submodules, one needs to find adequate patterns for <include>. In my case:

            <sourceDirectory>${basedir}</sourceDirectory>
            <includes>*.xml,src/**/*.xml,src/**/*.java,</includes>
            

            The reporter's problem should be solved by something like

            <sourceDirectory>${basedir}/src</sourceDirectory>
            <includes>*.*</includes>
            
            ppalaga Peter Palaga added a comment - I was about to vote this up but then I found a config that works for me and probably will work also for the reporter. My use case is that I want to check tabs and trailing spaces in pom.xml files. Simply setting <sourceDirectory>${basedir}</sourceDirectory> includes pom.xml but also pulls all submodule directories in, which I do not want. To get rid of the submodules, one needs to find adequate patterns for <include> . In my case: <sourceDirectory>${basedir}</sourceDirectory> <includes>*.xml,src /**/ *.xml,src /**/ *.java,</includes> The reporter's problem should be solved by something like <sourceDirectory>${basedir}/src</sourceDirectory> <includes>*.*</includes>
            rfscholte Robert Scholte added a comment - - edited

            How about src/main/* and src/test/*? There are other plugins which have there own folder and which should be verified as well, e.g. src/main/javadoc and src/main/sql.

            rfscholte Robert Scholte added a comment - - edited How about src/main/* and src/test/* ? There are other plugins which have there own folder and which should be verified as well, e.g. src/main/javadoc and src/main/sql .

            People

              Unassigned Unassigned
              lstrzelecki Łukasz Strzelecki
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: