Skip to content

Conversation

tltv
Copy link
Member

@tltv tltv commented Oct 9, 2025

Cleans up Maven build configuration by removing need for production profile. Updates Maven build configuration to run build-frontend goal by default on package phase. With help of spring-boot-maven-plugin feature, includeOptional=false, vaadin-dev dependency is not included in production package.

Part of vaadin/flow#22439

WIP NOTICE: Requires Spring Boot 4.0.0-SNAPSHOT and running requires Flow compatibility.

Cleans up Maven build configuration by removing need for production profile. Updates Maven build configuration to run build-frontend goal by default on package phase. With help of spring-boot-maven-plugin feature, includeOptional=false, vaadin-dev dependency is not included in production package.

Part of vaadin/flow#22439
<artifactId>spring-boot-maven-plugin</artifactId>
<configuration>
<!-- Set false to exclude optional dependencies like vaadin-dev. -->
<includeOptional>false</includeOptional>
Copy link
Member

@Artur- Artur- Oct 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this the default in Spring Boot 4?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, we probably end up removing this for v25 branches. For branches still using SB 3 (if we update those too) it will be true by default.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But this is a PR for v25?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean the moment when we have something else than SNAPSHOT to use and flow is updated too.

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.

2 participants