streamnative / pulsar-archived

Apache Pulsar - distributed pub-sub messaging system
https://pulsar.apache.org
Apache License 2.0
72 stars 25 forks source link

ISSUE-12271: [Improvement] Introduce Maven Modernizer plugin to Pulsar #3125

Open sijie opened 3 years ago

sijie commented 3 years ago

Original Issue: apache/pulsar#12271


As discussed in dev mail list: https://lists.apache.org/thread.html/re99b342765b2cfb26f3da0eedfb78e7312de158bf6c7ed0085d32781%40%3Cdev.pulsar.apache.org%3E We want to introduce Maven Modernizer plugin to Pulsar. This plugin helps detecting uses of legacy APIs which can be replaced with modern APIs that are often more performant, safer, and idiomatic than the legacy equivalents. We'll break down the task by sub-module so anyone interested can work on a sub-module independently. We'll start with checking violation with Java 8. Acceptance criteria will be passing existing CI. Help are welcome, please reply with module you want to work on so there won't be conflict. And if change for single module is too big, please break down pr to reasonable size, say < 100 change so it's easier to review. Please don't close the master issue via comment.

<plugin>
    <groupId>org.gaul</groupId>
    <artifactId>modernizer-maven-plugin</artifactId>
    <configuration>
        <failOnViolations>true</failOnViolations>
        <javaVersion>8</javaVersion>
    </configuration>
    <executions>
        <execution>
            <id>modernizer</id>
            <goals>
                <goal>modernizer</goal>
            </goals>
        </execution>
    </executions>
</plugin>
github-actions[bot] commented 2 years ago

The issue had no activity for 30 days, mark with Stale label.