Skip to content

refactor(@angular-devkit/build-angular): improve accuracy of programmatic watch mode usage for esbuild builders#25939

Merged
clydin merged 1 commit intoangular:mainfrom
clydin:esbuild/watch-diag-improvements
Oct 4, 2023
Merged

refactor(@angular-devkit/build-angular): improve accuracy of programmatic watch mode usage for esbuild builders#25939
clydin merged 1 commit intoangular:mainfrom
clydin:esbuild/watch-diag-improvements

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Oct 3, 2023

To better capture file changes after the initial build for the esbuild-based builders in a programmatic usage, the file watching initialization has been moved to before the first build results are yielded. This allows tests that execute code to change files with improved accuracy of the watch mode triggering. The application builder now also supports aborting the watch mode programmatically. This allows tests to gracefully stop the watch mode and more fully cleanup the test at completion.

…atic watch mode usage for esbuild builders

To better capture file changes after the initial build for the esbuild-based builders in a programmatic usage,
the file watching initialization has been moved to before the first build results are yielded. This allows tests
that execute code to change files with improved accuracy of the watch mode triggering. The application builder
now also supports aborting the watch mode programmatically. This allows tests to gracefully stop the watch mode
and more fully cleanup the test at completion.
@clydin clydin added target: major This PR is targeted for the next major release action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 3, 2023
@clydin clydin requested a review from alan-agius4 October 3, 2023 20:34
@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 Oct 4, 2023
@clydin clydin merged commit 9c4a6be into angular:main Oct 4, 2023
@clydin clydin deleted the esbuild/watch-diag-improvements branch October 4, 2023 13:56
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Nov 4, 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: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants