-
Notifications
You must be signed in to change notification settings - Fork 462
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
[3.2] Implement @at-root #799
Conversation
This is currently passing all the at-root specs. I'll do some clean but otherwise it's ready. |
8879cfe
to
34d6f55
Compare
return exactly<without_kwd>(src); | ||
} | ||
|
||
const char* foo(const char* src) { |
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.
Rename this.
a13dcbc
to
a674f97
Compare
I managed to rebase this post-sourcemaps-mega-merge with relatively little pain. I now have a failing at-root spec though. Once that's resolved I'll ship this. |
👍 Don't forget to check off any solved issues at #629 |
This is ready to go pending CI. I corrected some issues with nested output. |
Thanks for catching the output stuff @mgreter I amended that commit. That has also broken generic at-rule output (more than I expected). I was going to address at-rule bubbling and output in a different PR but I'll have to handle it here. |
f42dbcf
to
f3db16e
Compare
This PR implements
@at-root
.TODO
This feature is good to go.
Fixes #353. Specs added sass/sass-spec#175, sass/sass-spec#217, sass/sass-spec#219.
It turns out there were a lot of todo tests lying around addressing at-root. As a result the following specs now pass with this PR.