Skip to content

feat: BuildFrontend Incremental build#23884

Merged
mcollovati merged 13 commits intomainfrom
issues/17354-build-frontend-up-to-date
Mar 17, 2026
Merged

feat: BuildFrontend Incremental build#23884
mcollovati merged 13 commits intomainfrom
issues/17354-build-frontend-up-to-date

Conversation

@caalador
Copy link
Contributor

Add incremental build to the
buildFrontend task.

Closes #17354

Add incremental build to the
buildFrontend task.

Closes #17354
@github-actions
Copy link

github-actions bot commented Mar 16, 2026

Test Results

 1 387 files  ±0   1 387 suites  ±0   1h 30m 51s ⏱️ + 3m 8s
 9 923 tests ±0   9 852 ✅ ±0  71 💤 ±0  0 ❌ ±0 
10 396 runs  ±0  10 316 ✅ ±0  80 💤 ±0  0 ❌ ±0 

Results for commit 827d4fb. ± Comparison against base commit 03c9d39.

♻️ This comment has been updated with latest results.

@caalador caalador marked this pull request as ready for review March 16, 2026 09:47
Copy link
Collaborator

@mcollovati mcollovati left a comment

Choose a reason for hiding this comment

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

Would it be possible to add a couple of tests in MiscSingleModuleTest (similar to prepareFrontendIncrementalBuilds_featureEnabled and related) to ensure that a frontend build is performed only if an input change is detected or if the build is explicitly forced by the new alwaysExecuteBuildFrontend flag?

Also, do we need to take configuration cache into account, like in VaadinSmokeTest.testPrepareFrontend_configurationCache?

@caalador
Copy link
Contributor Author

Added tests.

@github-actions github-actions bot added +1.0.0 and removed +0.0.1 labels Mar 17, 2026
caalador and others added 4 commits March 17, 2026 09:11
BuildFrontend can not cache configurations
as it uses the Project Object as execution time
which is not allowed for caching configuration.
do not use prod bundle as the version
is set to 1111.5.0-SNAPSHOT or the like
@github-actions github-actions bot added +0.0.1 and removed +1.0.0 labels Mar 17, 2026
Fix WEBPACK to VITE
@sonarqubecloud
Copy link

@mcollovati mcollovati merged commit 88408e7 into main Mar 17, 2026
32 checks passed
@mcollovati mcollovati deleted the issues/17354-build-frontend-up-to-date branch March 17, 2026 14:08
vaadin-bot pushed a commit that referenced this pull request Mar 17, 2026
Add incremental build to the
buildFrontend task.

Closes #17354
@vaadin-bot
Copy link
Collaborator

Hi @caalador and @mcollovati, when i performed cherry-pick to this commit to 24.10, i have encountered the following issue. Can you take a look and pick it manually?
Error Message:
Error: Command failed: git cherry-pick 88408e7
error: could not apply 88408e7... feat: BuildFrontend Incremental build (#23884)
hint: After resolving the conflicts, mark them with
hint: "git add/rm ", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

@vaadin-bot
Copy link
Collaborator

Hi @caalador and @mcollovati, when i performed cherry-pick to this commit to 24.9, i have encountered the following issue. Can you take a look and pick it manually?
Error Message:
Error: Command failed: git cherry-pick 88408e7
error: could not apply 88408e7... feat: BuildFrontend Incremental build (#23884)
hint: After resolving the conflicts, mark them with
hint: "git add/rm ", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

vaadin-bot added a commit that referenced this pull request Mar 17, 2026
Add incremental build to the
buildFrontend task.

Closes #17354

Co-authored-by: caalador <mikael.grankvist@vaadin.com>
vaadin-bot added a commit that referenced this pull request Mar 17, 2026
Add incremental build to the
buildFrontend task.

Closes #17354

Co-authored-by: caalador <mikael.grankvist@vaadin.com>
caalador added a commit that referenced this pull request Mar 18, 2026
caalador added a commit that referenced this pull request Mar 18, 2026
mshabarov pushed a commit that referenced this pull request Mar 18, 2026
mshabarov pushed a commit that referenced this pull request Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Gradle Incremental Build for vaadinBuildFrontend task

3 participants