-
Notifications
You must be signed in to change notification settings - Fork 164
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: add method to skip clean #17080
Conversation
Add method to build-frontend plugin that makes it possible to skip cleaning of frontend files from extending plugins.
...-plugins/flow-maven-plugin/src/main/java/com/vaadin/flow/plugin/maven/BuildFrontendMojo.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A question: is the Gradle plugin also affected by a similar issue?
Yes, Gradle has the same functionality. Added the method there also so it can be overridden to not do the cleaning if necessary. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to add a test for this, but we can address it in another PR
flow-plugins/flow-gradle-plugin/src/main/kotlin/com/vaadin/gradle/VaadinBuildFrontendTask.kt
Outdated
Show resolved
Hide resolved
flow-plugins/flow-gradle-plugin/src/main/kotlin/com/vaadin/gradle/VaadinBuildFrontendTask.kt
Outdated
Show resolved
Hide resolved
flow-plugins/flow-gradle-plugin/src/main/kotlin/com/vaadin/gradle/VaadinBuildFrontendTask.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: Marco Collovati <marco@vaadin.com>
Kudos, SonarCloud Quality Gate passed! |
Add method to build-frontend
plugin that makes it possible
to skip cleaning of frontend
files from extending plugins.