Skip to content

fix(@angular-devkit/build-angular): use absolute watch paths for postcss dependency messages#25474

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
clydin:esbuild/absolute-postcss-deps
Jun 29, 2023
Merged

fix(@angular-devkit/build-angular): use absolute watch paths for postcss dependency messages#25474
alan-agius4 merged 1 commit intoangular:mainfrom
clydin:esbuild/absolute-postcss-deps

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Jun 29, 2023

Postcss's directory dependency message can use a glob pattern to reflect that multiple files within a given directory should be watched. When calculated the complete file set, the paths must be absolute to ensure that the watch system correctly invalidates the stylesheet being processed.

…css dependency messages

Postcss's directory dependency message can use a glob pattern to reflect that multiple files
within a given directory should be watched. When calculated the complete file set, the paths
must be absolute to ensure that the watch system correctly invalidates the stylesheet being
processed.
@clydin clydin added the target: patch This PR is targeted for the next patch release label Jun 29, 2023
@clydin clydin requested a review from alan-agius4 June 29, 2023 11:20
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Jun 29, 2023
@alan-agius4 alan-agius4 merged commit 376e346 into angular:main Jun 29, 2023
@clydin clydin deleted the esbuild/absolute-postcss-deps branch June 29, 2023 13:54
@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 Jul 30, 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.

2 participants