Uploaded image for project: 'Kylin'
  1. Kylin
  2. KYLIN-3310

Use lint for maven-compiler-plugin

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: Backlog
    • Component/s: Tools, Build and Test
    • Labels:
      None

      Description

      lint helps identify structural problems.

      We should enable lint for maven-compiler-plugin

                <artifactId>maven-compiler-plugin</artifactId>
                <version>${maven-compiler-plugin.version}</version>
                <configuration>
                  <source>1.8</source>
                  <target>1.8</target>
                  <compilerArgs>
                    <arg>-Xlint:all</arg>
                    <arg>${compiler.error.flag}</arg>
                    <!-- Override options warnings to support cross-compilation -->
                    <arg>-Xlint:-options</arg>
                    <!-- Temporary lint overrides, to be removed over time. -->
                    <arg>-Xlint:-cast</arg>
                    <arg>-Xlint:-deprecation</arg>
                    <arg>-Xlint:-processing</arg>
                    <arg>-Xlint:-rawtypes</arg>
                    <arg>-Xlint:-serial</arg>
                    <arg>-Xlint:-try</arg>
                    <arg>-Xlint:-unchecked</arg>
                    <arg>-Xlint:-varargs</arg>
                    <!-- Uncomment the following args to display more warnings. -->
                    <!-- -Xmaxwarns -->
                    <!-- 10000 -->
                  </compilerArgs>
                  <showWarnings>true</showWarnings>
                  <!-- Another temp override, to be set to true in due course. -->
                  <showDeprecation>false</showDeprecation>
                </configuration>
      

        Attachments

          Activity

            People

            • Assignee:
              Aron.tao Jiatao Tao
              Reporter:
              yuzhihong@gmail.com Ted Yu
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated: