yegor256 / qulice

Quality Police for Java projects: aggregator of Checkstyle and PMD
https://www.qulice.com
Other
296 stars 111 forks source link

Log4j warning in execution logs #509

Open mkordas opened 8 years ago

mkordas commented 8 years ago

Qulice execution logs contain log4j warnings:

[INFO] --- qulice-maven-plugin:0.14:check (default) @ template-project ---
[INFO] Starting Checkstyle validator
[INFO] LICENSE found: file:config/header.txt
log4j:WARN No appenders could be found for logger (com.puppycrawl.tools.checkstyle.ConfigurationLoader).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.

Logs should not be polluted with such warnings.

To reproduce the issue you can use simplest possible Maven project with Qulice:

<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
    <modelVersion>4.0.0</modelVersion>

    <groupId>any</groupId>
    <artifactId>any</artifactId>
    <version>any</version>
    <build>
        <plugins>
            <plugin>
                <groupId>com.qulice</groupId>
                <artifactId>qulice-maven-plugin</artifactId>
                <version>0.14</version>
                <configuration>
                    <license>file:${basedir}/header</license>
                </configuration>
                <executions>
                    <execution>
                        <phase>validate</phase>
                        <goals>
                            <goal>check</goal>
                        </goals>
                    </execution>
                </executions>
            </plugin>
        </plugins>
    </build>
</project>

and run mvn validate.

krzyk commented 8 years ago

@davvd valid bug

krzyk commented 8 years ago

@davvd this is postponed

davvd commented 8 years ago

@davvd valid bug

@krzyk I just added bug tag here

davvd commented 8 years ago

@davvd this is postponed

@krzyk right, I added "postponed" label

davvd commented 8 years ago

@davvd this is postponed

@krzyk I will try to find someone else

davvd commented 8 years ago

@mkordas thanks for tis bug, I topped your account for 15 mins, transaction 566aa93cd2b2cc0064000036