Skip to content

fix: Set required Java version to 1.8 in flow-maven-plugin#23398

Open
TatuLund wants to merge 2 commits into2.13from
flow-maven-plugin-java8
Open

fix: Set required Java version to 1.8 in flow-maven-plugin#23398
TatuLund wants to merge 2 commits into2.13from
flow-maven-plugin-java8

Conversation

@TatuLund
Copy link
Contributor

@TatuLund TatuLund commented Feb 3, 2026

Adapted from: vaadin/maven-plugin#176

@mcollovati
Copy link
Collaborator

I'm a bit confused: the documentation for maven-plugin-plugin says that requiredJavaVersion is evaluated by Maven 4 and ignored by earlier Maven versions..
Also, the property is marked Since: 3.8.0, but in the Flow 2.x branch the plugin version is 3.5.2

https://maven.apache.org/plugin-tools/maven-plugin-plugin/descriptor-mojo.html

@mcollovati
Copy link
Collaborator

So the check has been introduced also in Maven 3.9.12 (apache/maven#11479).

@TatuLund
Copy link
Contributor Author

TatuLund commented Feb 3, 2026

So the check has been introduced also in Maven 3.9.12 (apache/maven#11479).

Yes, it was just observed in one case with Vaadin 7

https://vaadin.com/forum/t/error-the-plugin-com-vaadin7-7-46-and-java-11-check/179171

Co-authored-by: Marco Collovati <marco@vaadin.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 3, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants