Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented Nov 13, 2023

When using the application and/or browser-esbuild builders, the current watch files
will be maintained in the event of a build error. This better ensures that files used
in the last build/rebuild that may have caused an error will continue to be watched and
refreshed when later changed. Only when a successful rebuild has occurred will any stale
watch files be removed from the watch list.

…r build errors

When using the `application` and/or `browser-esbuild` builders, the current watch files
will be maintained in the event of a build error. This better ensures that files used
in the last build/rebuild that may have caused an error will continue to be watched and
refreshed when later changed. Only when a successful rebuild has occurred will any stale
watch files be removed from the watch list.
@clydin clydin added the target: patch This PR is targeted for the next patch release label Nov 13, 2023
… application builder

When using the `application` and/or `browser-esbuild` builders, stylesheets
that generate errors will now have the errors cached between rebuilds during
watch mode (include `ng serve`). This not only avoids reprocessing files that
contain errors and that have not changed but also provides file watching information
from the cache to ensure the error-causing files are properly invalidated.
@clydin clydin force-pushed the esbuild/error-file-watching branch from d802fc2 to d0fb3b3 Compare November 13, 2023 23:30
@clydin clydin marked this pull request as ready for review November 14, 2023 00:03
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Nov 14, 2023
@clydin clydin requested a review from alan-agius4 November 14, 2023 00:03
@clydin clydin linked an issue Nov 14, 2023 that may be closed by this pull request
@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 Nov 15, 2023
@clydin clydin merged commit 447761e into angular:main Nov 15, 2023
@clydin clydin deleted the esbuild/error-file-watching branch November 15, 2023 12:54
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ng serve not detecting file changes
2 participants