Skip to content

[19.2.x]: Backport of Sass changes #30879

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 6 commits into from
Apr 16, 2025
Merged

Conversation

devversion
Copy link
Member

No description provided.

This will be much faster, doesn't rely on workers that execute outside
of Bazel sandbox and can cause subtle confusion (see recent Sass
changes).
This is cleaner and follows the new `tools/bazel/x.bzl` folder pattern.
Styling token extraction was included in docs-content because it was the
only way to conveniently expose data to the docs site. Now with the
proper monorepo setup, this is much easier and we can stop running this
JS-based Sass compilation that is rather slow— in development workflows.
Enables proper source maps in dev-app, and also includes sources in the
maps as most people probably don't integrate the source project into
dev tools.
We have an integration linker test that verifies the linker better.
The tests currently aren't verifying anything.
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Apr 16, 2025
@devversion devversion requested a review from alan-agius4 April 16, 2025 10:22
@devversion devversion added the target: patch This PR is targeted for the next patch release label Apr 16, 2025
@devversion devversion marked this pull request as ready for review April 16, 2025 10:22
@devversion devversion requested review from a team as code owners April 16, 2025 10:22
@devversion devversion requested review from adolgachev and mmalerba and removed request for a team April 16, 2025 10:22
@devversion devversion merged commit 9b25bce into angular:19.2.x Apr 16, 2025
24 of 26 checks passed
@devversion devversion deleted the patch-sass branch April 16, 2025 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: build & ci Related the build and CI infrastructure of the project 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