Skip to content
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

Rewrote at-root specs #175

Merged
merged 3 commits into from
Dec 30, 2014
Merged

Rewrote at-root specs #175

merged 3 commits into from
Dec 30, 2014

Conversation

KittyGiraudel
Copy link
Contributor

No description provided.

@KittyGiraudel
Copy link
Contributor Author

What is blocking here?

@xzyfer
Copy link
Contributor

xzyfer commented Dec 30, 2014

Nothing blocking this, just haven't had the time to look at it frankly.

It'd be worth adding some specs for with and without.

@KittyGiraudel
Copy link
Contributor Author

I got this.

@KittyGiraudel
Copy link
Contributor Author

Done. That should be enough.

@xzyfer
Copy link
Contributor

xzyfer commented Dec 30, 2014

Perfect! Thanks again

In future please make sure you generated the output with the nested style.

xzyfer added a commit that referenced this pull request Dec 30, 2014
@xzyfer xzyfer merged commit ee860fb into sass:master Dec 30, 2014
@smikes
Copy link

smikes commented Dec 31, 2014

I believe this should be marked as "TODO" for libsass, because at-root is not yet implemented. This makes the CI fail.

@mgreter
Copy link
Contributor

mgreter commented Dec 31, 2014

I've moved the at-root tests into the todo folder! I hope CI will now build correctly again!

@smikes
Copy link

smikes commented Dec 31, 2014

Thanks!

@xzyfer
Copy link
Contributor

xzyfer commented Dec 31, 2014

Sorry guys.

@KittyGiraudel
Copy link
Contributor Author

In future please make sure you generated the output with the nested style.

Apologies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants