airlift / airbase

Base POM for Airlift
Apache License 2.0
48 stars 87 forks source link

Update maven-dependency-plugin to 3.3.0 #304

Closed wendigo closed 2 years ago

wendigo commented 2 years ago

This version is compatible with JDK 16+:

Release Notes - Maven Dependency Plugin - Version 3.3.0

** Bug
* [MDEP-679] - mvn dependency:analyze detected wrong transitive
dependency
* [MDEP-742] - dependency plugin does not work with JDK 16
* [MDEP-752] - skip dependency analyze in ear packaging
* [MDEP-753] - Non-test dependency reported as Non-test scoped test
only dependency
* [MDEP-759] - 'Dependency not found' with 3.2.0 and Java-17 while
analyzing
* [MDEP-761] - Tree plugin does not terminate with 3.2.0
* [MDEP-769] - Minor improvement - continue
* [MDEP-774] - analyze-only failed: PermittedSubclasses requires ASM9
* [MDEP-781] - Broken Link to "Introduction to Dependency Mechanism
Page"
* [MDEP-783] - TreeMojo docs say scope doesn't work due to MSHARED-4
* [MDEP-786] - Sealed classes not supported

** New Feature
* [MDEP-787] - Allow ignoring non-test-scoped dependencies

** Improvement
* [MDEP-763] - Minor improvements
* [MDEP-768] - GitHub Action build improvement
* [MDEP-779] - dependency:analyze should list the classes that cause a
used undeclared dependency
* [MDEP-789] - Improve documentation of analyze - Non-test scoped

** Task
* [MDEP-760] - Java 1.8 as minimum

** Dependency upgrade
* [MDEP-766] - Upgrade maven-invoker-plugin to version 3.2.2
* [MDEP-784] - Upgrade maven-dependency-analyzer to 1.12.0
* [MDEP-788] - Upgrade maven-reporting-impl to version 3.1.0
* [MDEP-795] - Update Jetty to 9.4.45.v20220203
* [MDEP-796] - Upgrade Maven Parent to 35
* [MDEP-797] - Update transitive dependency commons-beanutils to 1.9.4
* [MDEP-798] - Upgrade maven-dependency-tree from 3.0.1 to 3.1.0
wendigo commented 2 years ago

cc @findepi @martint @electrum

findepi commented 2 years ago

This version is compatible with JDK 16+:

was the previous one not compatible with Java 17? i thought we already run full build with 17

wendigo commented 2 years ago

This version is compatible with JDK 16+:

was the previous one not compatible with Java 17?

i thought we already run full build with 17

@findepi: It's broken when language level >= 16. We are still targeting 11.

electrum commented 2 years ago

We use Java 17 in other projects and it works fine due to the dependency overrides. I need to examine the dependency scope changes in detail, since they don’t appear correct on first glance.

wendigo commented 2 years ago

@electrum can we merge it?

electrum commented 2 years ago

We can merge it.

wendigo commented 2 years ago

@electrum please do :)