Skip to content

Update maven-dependency-plugin to 3.3.0#304

Merged
electrum merged 1 commit intoairlift:masterfrom
wendigo:serafin/update-dependency-plugin
Mar 22, 2022
Merged

Update maven-dependency-plugin to 3.3.0#304
electrum merged 1 commit intoairlift:masterfrom
wendigo:serafin/update-dependency-plugin

Conversation

@wendigo
Copy link
Copy Markdown
Contributor

@wendigo wendigo commented Mar 18, 2022

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
Copy link
Copy Markdown
Contributor Author

wendigo commented Mar 18, 2022

cc @findepi @martint @electrum

@findepi
Copy link
Copy Markdown
Contributor

findepi commented Mar 18, 2022

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
Copy link
Copy Markdown
Contributor Author

wendigo commented Mar 18, 2022

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
Copy link
Copy Markdown
Member

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
Copy link
Copy Markdown
Contributor Author

wendigo commented Mar 21, 2022

@electrum can we merge it?

@electrum
Copy link
Copy Markdown
Member

We can merge it.

@wendigo
Copy link
Copy Markdown
Contributor Author

wendigo commented Mar 21, 2022

@electrum please do :)

@electrum electrum merged commit 334981e into airlift:master Mar 22, 2022
@wendigo wendigo deleted the serafin/update-dependency-plugin branch March 22, 2022 05:20
wendigo added a commit to wendigo/trino that referenced this pull request Apr 1, 2022
While testing maven-dependency-plugin update in airlift/airbase#304
it started reporting some dependencies as having invalid scope. These are now fixed and working both
on the old and new plugin versions.
hashhar pushed a commit to trinodb/trino that referenced this pull request Apr 4, 2022
While testing maven-dependency-plugin update in airlift/airbase#304
it started reporting some dependencies as having invalid scope. These are now fixed and working both
on the old and new plugin versions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants