Skip to content

fix(@angular/build): disable Worker wait loop for Sass compilations in web containers #27725

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

Merged
merged 1 commit into from
May 28, 2024

Conversation

alan-agius4
Copy link
Collaborator

The Sass Worker by default uses an Atomics-based wait loop to improve performance while waiting for messages. This loop relies on the synchronous API receiveMessageOnPort. While this works well in Node.js, the web container execution environment does not currently support passing transferable objects via receiveMessageOnPort.

Closes: #27723

…n web containers

The Sass Worker by default uses an Atomics-based wait loop to improve performance while waiting for messages. This
loop relies on the synchronous API `receiveMessageOnPort`. While this works well in Node.js, the web container
execution environment does not currently support passing transferable objects via `receiveMessageOnPort`.

Closes: angular#27723
@alan-agius4 alan-agius4 requested a review from clydin May 27, 2024 11:44
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels May 27, 2024
@alan-agius4 alan-agius4 marked this pull request as ready for review May 27, 2024 11:44
@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 May 28, 2024
@alan-agius4 alan-agius4 merged commit c9644c9 into angular:main May 28, 2024
34 checks passed
@alan-agius4 alan-agius4 deleted the sass-error branch May 28, 2024 09:03
JeanMeche added a commit to JeanMeche/angular that referenced this pull request Jun 1, 2024
Updating to CLI 18.0.2 will fix the Sass issue in the playgrounds (see angular/angular-cli#27725)
@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 Jun 28, 2024
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.

Angular 18 not working with SCSS defined
2 participants