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
build: make sass bazel targets consistent #14186
build: make sass bazel targets consistent #14186
Conversation
* Fixes that the `list.scss` accidentally imports the `divider.scss` (while it just wants an actual mixin) * Fixes that the `cdk/a11y` package does not expose the `a11y-prebuilt.css` as Bazel target * Makes all `BUILD` files consistent in regards to Sass targets. * Ensures that each `sass_binary` or `sass_library` has it's **explicit** dependencies. Currently it's not possible to depend on a single target (depending on transitive build outputs/targets) * Removes the `_partials` filegroups because those can be replaced with a `sass_library` now. Also removes the `theme` sass_library in favor of the `_scss_lib`. Note that this does not change anything for the `theming_bundle` or `core:all_themes` target.
Note: This is not merge safe because I fixed an issue with the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Caretaker Note: This will require BUILD file updates internally with the new file created. |
@@ -1,7 +1,7 @@ | |||
@import '../core/style/variables'; | |||
@import '../core/style/list-common'; | |||
@import '../core/style/layout-common'; | |||
@import '../divider/divider'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@crisbeto don't we have a lint check that should have caught this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The lint check only checks the other way around. It reports if you have static CSS within a file starting with an underscore. (e.g. _divider-offset.scss
)
* Fixes that the `list.scss` accidentally imports the `divider.scss` (while it just wants an actual mixin) * Fixes that the `cdk/a11y` package does not expose the `a11y-prebuilt.css` as Bazel target * Makes all `BUILD` files consistent in regards to Sass targets. * Ensures that each `sass_binary` or `sass_library` has it's **explicit** dependencies. Currently it's not possible to depend on a single target (depending on transitive build outputs/targets) * Removes the `_partials` filegroups because those can be replaced with a `sass_library` now. Also removes the `theme` sass_library in favor of the `_scss_lib`. Note that this does not change anything for the `theming_bundle` or `core:all_themes` target.
* Fixes that the `list.scss` accidentally imports the `divider.scss` (while it just wants an actual mixin) * Fixes that the `cdk/a11y` package does not expose the `a11y-prebuilt.css` as Bazel target * Makes all `BUILD` files consistent in regards to Sass targets. * Ensures that each `sass_binary` or `sass_library` has it's **explicit** dependencies. Currently it's not possible to depend on a single target (depending on transitive build outputs/targets) * Removes the `_partials` filegroups because those can be replaced with a `sass_library` now. Also removes the `theme` sass_library in favor of the `_scss_lib`. Note that this does not change anything for the `theming_bundle` or `core:all_themes` target.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
list.scss
accidentally imports thedivider.scss
(while it just wants an actual mixin)cdk/a11y
package does not expose thea11y-prebuilt.css
as Bazel targetBUILD
files consistent in regards to Sass targets.sass_binary
orsass_library
has it's explicit dependencies. Currently it's not possible to depend on a single target (depending on transitive build outputs/targets)_partials
filegroups because those can be replaced with asass_library
now. Also removes thetheme
sass_library in favor of the_scss_lib
. Note that this does not change anything for thetheming_bundle
orcore:all_themes
target.