Skip to content

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Dec 9, 2016

Fixes SASS errors when Material is imported directly.

Fixes #2133.

@crisbeto crisbeto requested a review from jelbourn December 9, 2016 08:43
@googlebot googlebot added the cla: no PR author must sign Google's Contributor License Agreement: https://opensource.google.com/docs/cla label Dec 9, 2016
Fixes SASS errors when Material is imported directly.

Fixes angular#2133.
@crisbeto crisbeto force-pushed the 2133/sass-path-fail branch from 6aac086 to b701e4d Compare December 9, 2016 08:47
@googlebot googlebot added cla: yes PR author has agreed to Google's Contributor License Agreement and removed cla: no PR author must sign Google's Contributor License Agreement: https://opensource.google.com/docs/cla labels Dec 9, 2016
@jelbourn
Copy link
Member

jelbourn commented Dec 9, 2016

LGTM

I'd like to figure out why this didn't cause a problem: our build worked, sample apps worked, and all of the internal Google sass builds using Material worked. Is it possibly something that is only an error in older versions of sass? Is it a ruby vs. libsass thing?

@jelbourn jelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Dec 9, 2016
@crisbeto
Copy link
Member Author

crisbeto commented Dec 9, 2016

No idea, I can spend some more time with it over the weekend.

@mmalerba mmalerba merged commit 9ed29fe into angular:master Dec 9, 2016
@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 Sep 6, 2019
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 cla: yes PR author has agreed to Google's Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some SCSS files refer to invalid path (a11y)
4 participants