Skip to content
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

Update gradle plugin VaadinCleanTask to match maven clean-frontend target #18556

Closed
caalador opened this issue Jan 29, 2024 · 1 comment · Fixed by #18756
Closed

Update gradle plugin VaadinCleanTask to match maven clean-frontend target #18556

caalador opened this issue Jan 29, 2024 · 1 comment · Fixed by #18756

Comments

@caalador
Copy link
Contributor

Describe your motivation

Both plugins should clean files in the same way and have the same configuration possibilities.
At the moment for extending the clean task maven has the option to disable removal/cleaning of files and folders where as
gradle doesn't.

Also the gradle task is missing the cleanup of framework manages packages from package.json

Describe the solution you'd like

Cleanup code should possibly be moved into plugin-base and both maven and gradle would call this cleanup code instead of having their own implementations.

Describe alternatives you've considered

Gradle task should be updated to do the same cleanup as its maven counterpart.

@tltv tltv self-assigned this Feb 19, 2024
tltv added a commit that referenced this issue Feb 19, 2024
Updates Vaadin Gradle plugin to clean in same way as Maven plugin cleans. Moved  CleanFrontendMojo logic to CleanFrontendUtil in shared flow-plugin-base.
Updated VaadinCleanTask and VaadinBuildFrontendTask.
Added cleanFrontendFiles property to Gradle plugin.

Fixes: #18556
mshabarov pushed a commit that referenced this issue Feb 22, 2024
* feat: update gradle plugin clean tasks

Updates Vaadin Gradle plugin to clean in same way as Maven plugin cleans. Moved  CleanFrontendMojo logic to CleanFrontendUtil in shared flow-plugin-base.
Updated VaadinCleanTask and VaadinBuildFrontendTask.
Added cleanFrontendFiles property to Gradle plugin.

Fixes: #18556

* test: added tests

* chore: use local stub jar

...instead of using real hilla endpoint. Also removed webpack.generated.js from the task description and javadoc.

* chore: extract Options to CleanOptions

* docs: added javadoc
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.4.0.

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

Successfully merging a pull request may close this issue.

4 participants