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

fix(@angular/build): support incremental build file results in watch mode #29191

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

clydin
Copy link
Member

@clydin clydin commented Dec 19, 2024

When the application build is in watch mode, incremental build results will now be generated. This allows fine-grained updates of the files in the output directory and supports removal of stale application code files. Note that stale assets will not currently be removed from the output directory. More complex asset change analysis will be evaluated for inclusion in the future to address this asset output behavior.

Closes #26595

@clydin clydin added the target: minor This PR is targeted for the next minor release label Dec 19, 2024
@clydin clydin force-pushed the application/incremental-results branch 7 times, most recently from 93dd029 to d624b13 Compare December 20, 2024 01:18
@clydin clydin marked this pull request as ready for review December 20, 2024 01:57
…mode

When the application build is in watch mode, incremental build results will
now be generated. This allows fine-grained updates of the files in the output
directory and supports removal of stale application code files.
Note that stale assets will not currently be removed from the output directory.
More complex asset change analysis will be evaluated for inclusion in the future
to address this asset output behavior.
@clydin clydin force-pushed the application/incremental-results branch from d624b13 to 119283c Compare January 2, 2025 16:49
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jan 2, 2025
@clydin clydin requested a review from alan-agius4 January 2, 2025 17:41
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 6, 2025
@alan-agius4 alan-agius4 merged commit ba16ad6 into angular:main Jan 6, 2025
32 checks passed
@clydin clydin deleted the application/incremental-results branch January 6, 2025 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: @angular/build target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

watch did not clean up old files
2 participants